Repository navigation
chore: add semver-major ignore for astro in Dependabot config - #190
Conversation
@astrojs/starlight@0.40.0 declares peerDependencies: { astro: "^6.4.5" }
and no Starlight version supporting Astro 7 exists. Dependabot PR #188
bumped astro to 7.0.0, causing npm ci to fail with ERESOLVE. Major Astro
upgrades require coordinated ecosystem support and should be manually
initiated after readiness is confirmed.
Closes #188
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details🔇 Additional comments (1)
Summary by CodeRabbit
WalkthroughThis change updates the Dependabot configuration for GitHub Actions so the existing ChangesDependabot GitHub Actions configuration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Warning Review ran into problems🔥 ProblemsLinked repositories: Your configuration references 2 linked repositories, but your current plan allows 0. Analyzed ``, skipped 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 |
JasonCust
left a comment
There was a problem hiding this comment.
Lead Architect Review
Verdict: APPROVE (posted as COMMENT — GitHub prevents self-approval)
Surgically correct single-line configuration change that prevents doomed Dependabot PRs for Astro major versions until the Starlight ecosystem supports the target version. Root cause clearly identified (#188 failure mode), DECISIONS.md already documents the decision with removal condition, and tracking issue #189 exists for the deliberate migration.
- Blocking items: 0
- Non-blocking items: 0
|
@coderabbitai full review |
✅ Action performedFull review finished. |
Summary
version-update:semver-majorto the Dependabot ignore list forastroin.github/dependabot.ymlContext
PR #188 (Dependabot astro 7.0.0 bump) failed CI because
@astrojs/starlight@0.40.0declarespeerDependencies: { astro: "^6.4.5" }— no Starlight version supports Astro 7. The existingsemver-minorignore rule prevented minor bumps but allowed the major bump through.Related
@ascorbic/loader-utils— verified still needed, no change)Test plan
npm run buildsucceeds (169 pages, exit 0)