Repository navigation
dependabot - keep astro/@astrojs packages out of catch-all groups - #82
Merged
Merged
Conversation
There was a problem hiding this comment.
馃煝 Approval recommended
The exclusions match the existing Astro group patterns and leave other update policies unchanged.
No open issues 馃帀
What changed in this PR
This PR makes Dependabot鈥檚 catch-all groups exclude Astro packages, keeping updates for the documentation site鈥檚 Astro stack in the dedicated astro group.
Changes:
- Excludes
astroand@astrojs/*from both production and development dependency groups. - Adds a comment explaining the exclusions and referencing the previously split update PRs.
| File | Description |
|---|---|
.github/鈥媎ependabot.yml |
Adds matching Astro exclusions to both catch-all dependency groups. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
astrogroup is meant to keep Astro and its first-party@astrojs/*packages upgrading together, but listing it first wasn't enough. Dependabot still put some of them in the catch-all groups:astroin the astro group (as intended)@astrojs/starlightin production-dependencies@astrojs/checkin dev-dependenciesThis adds
exclude-patternsforastroand@astrojs/*to the production-dependencies and dev-dependencies groups, so those packages can only land in the astro group.After merging, Dependabot should close #80 and #81 and fold those bumps into the astro group PR on its next run.