-
Notifications
You must be signed in to change notification settings - Fork 2
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
Version Packages #536
Version Packages #536
Conversation
GeraldRequired Reviewers
Don't want to be involved in this pull request? Comment |
c5bc5e0
to
a8cc119
Compare
Size Change: 0 B Total Size: 4.32 kB ℹ️ View Unchanged
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #536 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 59 59
Lines 839 839
Branches 233 233
=========================================
Hits 839 839 Continue to review full report at Codecov.
|
7a4b447
to
c5734ce
Compare
## Summary: When reviewing the changes in #536, I noticed that some packages were being bumped a major version even though none of the changeset .md files specified such changes. It appears this is happening due changeset treating peer deps differently from regular deps. This PR fixes the issue by make wonder-stuff-core a regular dependency of wonder-stuff-server. Issue: None ## Test plan: - land this PR and see #536 update appropriately Author: kevinbarabash Reviewers: jeresig, #frontend-infra, #typescript, somewhatabstract Required Reviewers: Approved By: jeresig Checks: ✅ codecov/project, ✅ Test (macos-latest, 16.x), ✅ CodeQL, ✅ Lint, typecheck, and coverage check (ubuntu-latest, 16.x), ✅ Prime node_modules cache for primary configuration (ubuntu-latest, 16.x), ✅ Analyze (javascript), ✅ gerald, ⏭ dependabot Pull Request URL: #544
c5734ce
to
a770367
Compare
## Summary: I updated wonder-stuff-server's deps in #544, but missing that wonder-stuff-testing and wonder-stuff-server-google are affected by the same issue. This PR fixes them as well. Issue: None ## Test plan: - Land and wait for #536, see that wonder-stuff-server-google and wonder-stuff-testing are bumped by a minor version Author: kevinbarabash Reviewers: jeresig Required Reviewers: Approved By: jeresig Checks: ✅ codecov/project, ✅ Test (macos-latest, 16.x), ✅ CodeQL, ✅ Lint, typecheck, and coverage check (ubuntu-latest, 16.x), ✅ Prime node_modules cache for primary configuration (ubuntu-latest, 16.x), ✅ gerald, ✅ Analyze (javascript), ⏭ dependabot Pull Request URL: #545
a770367
to
857e24a
Compare
857e24a
to
2d8d059
Compare
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.
All of the version bumps are looking good now. 🎉
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
@khanacademy/[email protected]
Minor Changes
Patch Changes
[email protected]
Minor Changes
Patch Changes