-
Notifications
You must be signed in to change notification settings - Fork 105
Update dependabot and release template configuration #720
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
Update dependabot and release template configuration #720
Conversation
WalkthroughDependabot configuration removes the skip-changelog label from GitHub Actions and gomod updates. The release draft template adds the dependencies label to categorized labels and removes replacers that filtered @dependabot-preview and @dependabot mentions. No scheduling or rebase changes; remaining template replacers stay unchanged. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor Dev as Developer
participant DB as Dependabot
participant GH as GitHub PR
participant RD as Release Draft
Note over DB,GH: Update PR creation (labels adjusted)
DB->>GH: Open PR with label "dependencies"<br/>(no "skip-changelog")
GH-->>Dev: PR visible with labels
Note over GH,RD: Release draft generation
RD->>GH: Read merged PRs and labels
alt Label-based categorization
RD->>RD: Include under "dependencies" category
end
Note right of RD: No longer stripping @dependabot mentions
RD-->>Dev: Draft release notes with preserved mentions
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #720 +/- ##
=======================================
Coverage 85.98% 85.98%
=======================================
Files 22 22
Lines 3854 3854
=======================================
Hits 3314 3314
Misses 383 383
Partials 157 157 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.github/dependabot.yml(0 hunks).github/release-draft-template.yml(1 hunks)
💤 Files with no reviewable changes (1)
- .github/dependabot.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: integration-tests (go latest version)
- GitHub Check: integration-tests (go current version)
| label: | ||
| - 'dependencies' | ||
| - 'maintenance' |
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.
🧩 Analysis chain
Fix key: use "labels" (plural) when providing multiple labels.
Release Drafter expects "labels" for an array. Using "label" with a list may cause this category to be ignored, and dependency PRs might not appear under "Maintenance/misc".
Apply this diff:
- title: '⚙️ Maintenance/misc'
- label:
+ labels:
- 'dependencies'
- 'maintenance'
- 'documentation'🌐 Web query:
In Release Drafter's configuration, should category entries use "labels" (array) instead of "label" when specifying multiple labels?
💡 Result:
Yes — use labels (an array) when specifying multiple labels. The official config examples and docs show categories using a labels: [ ... ] list; label (singular) is for a single value and using the wrong form can cause unexpected behavior. [1][2]
Use "labels" (plural) for Release Drafter category labels.
Release Drafter expects a labels: (array); using label: with multiple values can cause the category to be ignored.
File: .github/release-draft-template.yml (lines 20-22)
Apply this diff:
- title: '⚙️ Maintenance/misc'
- label:
+ labels:
- 'dependencies'
- 'maintenance'
- 'documentation'📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| label: | |
| - 'dependencies' | |
| - 'maintenance' | |
| - title: '⚙️ Maintenance/misc' | |
| labels: | |
| - 'dependencies' | |
| - 'maintenance' | |
| - 'documentation' |
🤖 Prompt for AI Agents
.github/release-draft-template.yml around lines 20 to 22: the template uses
"label:" (singular) with multiple values which Release Drafter ignores; change
the key to "labels:" (plural) so the array of category labels is recognized,
keeping the existing list items intact.
This PR is auto-generated.
This PR makes two changes:
These changes improve dependency management and release categorization.
Summary by CodeRabbit