Skip to content

fix: combine Babel 8 updates and single-PR Dependabot grouping - #123

Merged
jchrist merged 1 commit into
mainfrom
fix/babel-8-combined-and-dependabot-single-pr
Sep 29, 2026
Merged

jchrist merged 1 commit into
mainfrom
fix/babel-8-combined-and-dependabot-single-pr

Conversation

@jchrist

@jchrist jchrist commented Sep 29, 2026

Copy link
Copy Markdown
Owner

Supersedes #120 and #121.

Problem: #120 bumps @babel/core 7.29.6→8.0.6 alone, #121 bumps @babel/preset-env 7.29.5→8.0.6 alone. Each leaves the other on v7, whose peer dep @babel/core ^7.0.0-0 conflicts with core 8, so npm ci fails with ERESOLVE and both Check dist/ and JavaScript Tests fail.

Fix:

  • Bump both together to ^8.0.6 in package.json + lockfile. Verified locally: npm ci, format:check, lint, ci-test (15 passed), svg, package all pass. dist/index.js unchanged (rollup build does not use babel).
  • Simplify .github/dependabot.yml to one group per ecosystem (all-actions, all-npm-dependencies with patterns: ['*']) so all updates including majors arrive in a single PR per ecosystem. Cross-ecosystem single PR is not supported by Dependabot.

After merge, #120 and #121 should be closed as superseded.

- Bump @babel/core and @babel/preset-env together to ^8.0.6.
  Individual Dependabot PRs #120 and #121 each bump one package,
  leaving the other on v7 which peer-depends on core ^7 and breaks
  npm ci with ERESOLVE.
- Simplify .github/dependabot.yml to a single group per ecosystem
  (all-actions, all-npm-dependencies with patterns ['*']) so future
  updates, including majors, arrive as one PR.
@jchrist
jchrist merged commit 8e82ae6 into main Sep 29, 2026
6 checks passed
@jchrist
jchrist deleted the fix/babel-8-combined-and-dependabot-single-pr branch September 29, 2026 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant