ci: enable trusted publishing #29
Conversation
|
|
Warning Rate limit exceeded@JounQin has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 15 minutes and 3 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests
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. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
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. |
![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 1a64ce0 in 25 seconds. Click for details.
- Reviewed
20lines of code in1files - Skipped
0files when reviewing. - Skipped posting
2draft 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. .github/workflows/release.yml:34
- Draft comment:
Added npm upgrade step: confirm this is necessary and won't conflict with Node's bundled npm. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%None
2. .github/workflows/release.yml:50
- Draft comment:
Removed NPM token env vars: ensure trusted publishing still works without them. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%None
Workflow ID: wflow_KVXR43Gi27mmecqm
You can customize 
There was a problem hiding this comment.
Pull Request Overview
This PR enables trusted publishing for package releases by modifying the GitHub Actions release workflow. The changes remove the need for manually managed NPM tokens by leveraging GitHub's trusted publishing mechanism.
- Adds npm upgrade step to ensure latest npm version with trusted publishing support
- Removes manual NPM token configuration in favor of automated trusted publishing
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
📊 Package size report No changes
Unchanged files
🤖 This report was automatically generated by pkg-size-action |
|
Deploy preview for eslint-import-context ready! ✅ Preview Built with commit 1a64ce0. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #29 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 52 52
Branches 16 16
=========================================
Hits 52 52 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
size-limit report 📦
|
commit: |
Important
Enhances release workflow by upgrading npm to the latest version and removing specific npm environment variables.
.github/workflows/release.yml.NPM_CONFIG_PROVENANCEandNPM_TOKENenvironment variables from the release job in.github/workflows/release.yml.This description was created by
for 1a64ce0. You can customize this summary. It will automatically update as commits are pushed.