-
Notifications
You must be signed in to change notification settings - Fork 8
chore: use trust publish #27
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
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Caution Review failedThe pull request is closed. Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughGitHub Actions workflows updated: nodejs.yml adds merge_group trigger and bumps Node.js version from 23 to 24; release.yml renames job to NPM, switches to npm-release.yml workflow, and removes NPM_TOKEN secret. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
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. Comment |
commit: |
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 updates GitHub Actions workflows to modernize the release process and update Node.js test versions.
- Migrates the release workflow from
node-release.ymltonpm-release.ymland updates the job name accordingly - Updates Node.js test matrix to use version 24 instead of version 23
- Removes the
NPM_TOKENsecret from the release workflow configuration
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/release.yml | Switches to npm-release workflow and removes NPM_TOKEN from secrets |
| .github/workflows/nodejs.yml | Updates Node.js test version from 23 to 24 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #27 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 117 117
Branches 30 30
=========================================
Hits 117 117 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary by CodeRabbit
Tests
Chores