fix: v4 should use latest npm tag #214
Conversation
🦋 Changeset detectedLatest commit: e3f2112 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Pull Request Overview
This PR updates the release script in package.json to remove the explicit "--tag release-v4" flag so that the package now uses the latest npm tag by default.
- Removed the explicit tag flag from the "release" script command.
- Updated package.json to allow changeset publish to use the default tag.
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to 84ed20e in 28 seconds. Click for details.
- Reviewed
13lines of code in1files - Skipped
0files when reviewing. - Skipped posting
1draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:65
- Draft comment:
Removing '--tag release-v4' now defaults the tag to 'latest'. Ensure all downstream release docs and processes are updated accordingly. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%The comment is informative and suggests ensuring that documentation and processes are updated, which violates the rules against making purely informative comments and asking the author to ensure something.
Workflow ID: wflow_XAjwL6A2zE28zNC2
You can customize 
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 4.x #214 +/- ##
=======================================
Coverage 91.61% 91.61%
=======================================
Files 9 9
Lines 167 167
Branches 36 36
=======================================
Hits 153 153
Misses 12 12
Partials 2 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Important
Update
releasescript inpackage.jsonto use default npm tag instead ofrelease-v4.releasescript inpackage.jsonto use default npm tag instead ofrelease-v4.This description was created by
for 84ed20e. You can customize this summary. It will automatically update as commits are pushed.