chore: tighten dependabot npm grouping to minor/patch updates#184
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the Dependabot configuration to group npm updates by 'minor' and 'patch' versions across different directories. The reviewer suggests using more descriptive, directory-specific group names to avoid ambiguous pull request titles in the GitHub UI.
| all-npm-minor-patch: | ||
| patterns: | ||
| - "*" | ||
| update-types: | ||
| - "minor" | ||
| - "patch" |
There was a problem hiding this comment.
While the current configuration correctly restricts the group to minor and patch updates, using the same group name all-npm-minor-patch across multiple package-ecosystem entries can lead to ambiguous pull request titles in the GitHub UI. Dependabot typically uses the group name in the PR title (e.g., Bump all-npm-minor-patch dependencies). To improve triage efficiency and clarity, consider using directory-specific group names.
dashboard-npm-minor-patch:
patterns:
- "*"
update-types:
- "minor"
- "patch"| all-npm-minor-patch: | ||
| patterns: | ||
| - "*" | ||
| update-types: | ||
| - "minor" | ||
| - "patch" |
There was a problem hiding this comment.
Similar to the dashboard configuration, using a more specific group name here (e.g., showcase-npm-minor-patch) would help distinguish these grouped updates from those in other directories when viewing the repository's pull request list.
showcase-npm-minor-patch:
patterns:
- "*"
update-types:
- "minor"
- "patch"
Summary:
/dashboard/appand/showcase/appby renaming the group toall-npm-minor-patchand addingupdate-types: ["minor","patch"]so major npm updates will not be grouped into the shared npm PRs.Changed files:
.github/dependabot.ymlTesting:
python scripts/run_checks.py; it completed successfully and wrotedocs/reports/check-report.md, which was intentionally reverted so this PR remains config-only.git status --shortthat only.github/dependabot.ymlwas modified and committed.Risks:
Next:
Codex Task