-
Notifications
You must be signed in to change notification settings - Fork 0
fix(changesets): added missing dependency #142
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes a missing dependency issue by adding @changesets/changelog-github to the project's devDependencies. The package was already configured in .changeset/config.json but wasn't actually installed. Additionally, the PR removes the automatic prettier formatting of changelog files from the version script, as the team plans to implement a custom changelog formatting function in the future.
Key Changes
- Added missing
@changesets/changelog-github@0.5.2dependency - Removed prettier formatting step from
changeset:versionscript - Improved JSON formatting in
.changeset/config.jsonfor better readability
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Added @changesets/changelog-github to devDependencies and removed prettier formatting from the changeset:version script |
| .changeset/config.json | Reformatted changelog configuration array for better readability (multi-line format) |
| pnpm-lock.yaml | Auto-generated lockfile updates reflecting the new dependency and its transitive dependencies (dotenv, dataloader, node-fetch, etc.) |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
🚀 Performance Benchmark Results
✅ All benchmarks passed! |
Description
Added missing dependency.
And remove previous approach for prettification again, as we seem to need to write a changelog-function on our own.Type of Change
Testing
Checklist
Browser Testing
If applicable, please test in the following browsers:
Performance Impact
If this change affects performance:
Breaking Changes
If this introduces breaking changes, please describe:
Additional Notes
Any additional information that reviewers should know.