Repository navigation
Conversation
📝 WalkthroughWalkthroughThe PR updates the Changesets configuration to use the standard ChangesChangesets Changelog Generator Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.changeset/config.json:
- Around line 4-5: The config entry for "`@changesets/changelog-github`" uses the
unsupported option "disableThanks"; update the .changeset/config.json entry to
remove "disableThanks" or replace it with the supported "exclude" object (keys:
"user", "pr", "commit") so suppression of "Thanks …" works with
`@changesets/changelog-github`@0.7.0—e.g., locate the
"`@changesets/changelog-github`" array item and either drop the disableThanks
property or convert it into an appropriate "exclude" object.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 7fd2ab03-acec-4c09-bded-f83b11815866
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (2)
.changeset/config.jsonpackage.json
|
I checked the published Could you refresh this branch against current main and regenerate the lockfile with the repo's pinned pnpm? The current diff also changes the Changesets Node types peer from 24.6.0 to 22.15.3 and drops jsdom and Vite's lightningcss, terser, and tsx peers from the workspace test resolution. Please keep the lockfile change scoped to the changelog generator and its dependency graph. For the refreshed diff, please also run a disposable versioning dry run against a representative changeset so we can verify the rendered changelog and version bumps. The status command alone does not exercise the generator. |
Summary
Migrate the Changesets changelog generator from the compact community package to the official GitHub changelog package.
Changes
.changeset/config.jsonto use@changesets/changelog-github.repooption and addeddisableThanks: true.@changesets/changelog-githubat^0.7.0.pnpm-lock.yamlwith pnpm.Notes
Future changelog entries will use the official Changesets GitHub layout instead of the compact suffix layout.
Verification
git grep -n "@svitejs/changesets-changelog-github-compact"returned no matches.git grep -n "@changesets/changelog-github" -- .changeset/config.json package.json pnpm-lock.yamlconfirmed config, root dev dependency, and lockfile entries.node -e "import('@changesets/changelog-github').then((m)=>{ if (typeof m.default?.getReleaseLine !== 'function' || typeof m.default?.getDependencyReleaseLine !== 'function') throw new Error('missing changelog functions'); console.log('ok changelog exports') })"passed.pnpm changeset status --since=mainpassed and reported no packages to be bumped.Summary by CodeRabbit