-
Notifications
You must be signed in to change notification settings - Fork 0
Add monthly cron workflow for label updates #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add monthly cron workflow for label updates #27
Conversation
WalkthroughA new GitHub Actions workflow, "Monthly Label Timer," has been added. This workflow runs monthly and on manual trigger, prevents overlapping runs, and calls a reusable label update workflow from the Changes
Sequence Diagram(s)sequenceDiagram
participant GitHub Scheduler
participant Monthly Label Timer Workflow
participant escendit/keystone Utility Workflow
GitHub Scheduler->>Monthly Label Timer Workflow: Trigger (cron or manual)
Monthly Label Timer Workflow->>Monthly Label Timer Workflow: Enforce concurrency, set permissions
Monthly Label Timer Workflow->>escendit/keystone Utility Workflow: Invoke reusable workflow with secrets
escendit/keystone Utility Workflow-->>Monthly Label Timer Workflow: Update labels
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Assessment against linked issues
Poem
Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. ✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.github/workflows/cron-update-labels.yml (1)
11-15: Concurrency LGTM; consider a more generic group key.Using github.workflow improves future-proofing if the workflow name changes or is reused.
Apply this minimal change:
- group: labels-monthly-${{ github.ref }} + group: ${{ github.workflow }}-${{ github.ref }}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/cron-update-labels.yml(1 hunks)
🔇 Additional comments (4)
.github/workflows/cron-update-labels.yml (4)
1-2: Meets the objective (monthly label sync via reusable workflow).Title and purpose are clear and aligned with issue #26.
7-9: Validate the cron timing and timezone.Cron runs at 01:29 UTC on the 1st of each month. Confirm this matches your intended local time and cadence.
16-20: Permissions look minimal; confirm they match the called workflow’s needs.issues: write should be sufficient for label management. Verify the reusable workflow doesn’t require contents: write or additional scopes (e.g., pull-requests: write).
23-26: ✔ Reusable workflow call verified
- The file
escendit/keystone/.github/workflows/utility-update-labels.ymlexists onmain.- It declares
on: workflow_call, making it callable.- It defines
permissions: issues: write, which covers all label-creation/update operations.- Passing
secrets: inheritwill supply the requiredGITHUB_TOKEN.No further changes needed.
Closes #26
Summary by CodeRabbit