chore(ci): add .github/dependabot.yml — stop recurring astro-major PR flood (closes #123) - #124
Draft
beeeku wants to merge 1 commit into
Draft
chore(ci): add .github/dependabot.yml — stop recurring astro-major PR flood (closes #123)#124beeeku wants to merge 1 commit into
beeeku wants to merge 1 commit into
Conversation
…loses #123) Adds an explicit Dependabot config that stops the recurring astro-major PR churn against apps/docs (5 recurrences: #110, #116, #119, #121, #122) and pre-empts the same pattern for vitest majors across the workspace, both of which are blocked on coordinated migrations tracked in #115 and #120 respectively. Two directory entries mirror the repo's dependency graph: - /apps/docs — its own manifest with astro + starlight; ignores astro majors, groups patch/minor into a single npm_and_yarn PR. - / — Bun workspace root that resolves packages/*, integrations/*, and tooling/* via the shared root lockfile; ignores vitest majors. Divergence from the issue's proposed yaml: the vitest ignore moved from the apps/docs block to the root block, because apps/docs does not declare vitest — the ignore would never have fired there. Vitest lives in the root manifest and packages/*/@workkit/vitest-config, so the root entry is where the ignore takes effect. Each ignore entry carries an inline pointer to its tracker issue so it is obvious when the entry becomes removable.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Contributor
Bundle Size Report
|
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.
Summary
Closes #123. Adds an explicit
.github/dependabot.ymlso Dependabot stops re-opening the same Astro-major bump againstapps/docsevery couple of weeks (five recurrences so far: #110, #116, #119, #121, #122) — every one blocked on the same Starlight peer-dep wall tracked in #115. Same treatment pre-empts the workspace-wide Vitest v3 → v4 tracker in #120.What ships
.github/dependabot.ymlwith two directory entries that mirror the repo's actual dependency graph:/apps/docs— its own manifest withastro+@astrojs/starlight+@astrojs/tailwind. Groups patch/minor into the existing implicitnpm_and_yarngroup PR, ignoresastromajors until docs: Astro 6 upgrade tracker (apps/docs) — blocked on Starlight #115 lands./— Bun workspace root; the shared root lockfile resolves everypackages/*,integrations/*, andtooling/*transitive dep. Ignoresvitestmajors until chore(deps): Vitest 4 upgrade tracker — workspace-wide migration from v3 #120 lands.Both ignore entries carry an inline comment pointing to their tracker issue, so the entry becomes obviously removable once the tracked migration lands.
One divergence from #123's proposed yaml
The issue's spec placed the
vitestignore under/apps/docs. I moved it to the root block after confirmingapps/docs/package.jsonhas novitestdependency — an ignore there would never fire. Vitest lives in the root manifest andpackages/*/devDependenciesvia@workkit/vitest-config, so/is where the ignore actually takes effect. Same behavioral outcome as the acceptance criteria demand: no new Vitest-major PRs until #120 removes the entry.Sanity-checks from #123
.github/dependabot.yml. Adding one narrows Dependabot to the enumerated directories, which is exactly the intent. Root/and/apps/docsare the only two manifests where new-dep-version PRs are meaningful; workspace packages inherit through the root lockfile.npm_and_yarngroup conflict with the auto-generated group name Dependabot has been using?" The group name matches what Dependabot has been emitting in the PR titles (npm_and_yarn group across N directories), so tracker automations keying on that name should keep working. Only/apps/docsdeclares the group here; the root block leaves per-dep PRs alone since there is no equivalent recurrence pattern to consolidate on that side.Acceptance criteria (from #123)
.github/dependabot.ymladded with the two ignore entries (astro major on/apps/docs, vitest major on/).apps/docsnpm_and_yarngroup — verifiable on the next Dependabot run.Follow-ups this unblocks
.github/dependabot.yml" step as part of their close-out, so future me does not forget the migration is done.Test plan
bun run constitution:check --diff-only --base=origin/master(the diff is a single top-level config file — nopackages/*/src/**orapps/*/src/**touched, so constitution gates should be a no-op).github/dependabot.ymlis 43 lines, well-formed YAML with only well-known Dependabot keys)·@·d·ependabot i·gnore t·his major versionin the meantime.Generated by Claude Code