chore: remove outdated OTP release scripts #11885
Conversation
github-actions
bot
added
the
team
There was a problem hiding this comment.
Pull Request Overview
This PR removes outdated OTP (One-Time Password) authentication scripts and dependencies following Rspack's migration to npm trusted publishing. The changes eliminate the --otp command-line option, remove the OTP-specific publish function, and update documentation to reflect the new release workflow.
Key Changes:
- Removed OTP authentication logic and retry mechanism from the publish script
- Removed
@continuous-auth/clientdependency from package.json - Updated release documentation to reference the new workflow without OTP
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| x.mjs | Removed --otp command-line option from publish command |
| scripts/release/publish.mjs | Removed OTP-specific publish function and simplified to single publish method |
| package.json | Removed @continuous-auth/client dependency |
| website/docs/en/contribute/development/releasing.md | Updated workflow reference from Release Full(OTP) to Release Full |
| website/docs/zh/contribute/development/releasing.md | Updated workflow reference from Release Full(OTP) to Release Full (Chinese version) |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
✅ Deploy Preview for rspack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📦 Binary Size-limit
🙈 Size remains the same at 47.66MB |
CodSpeed Performance ReportMerging #11885 will not alter performanceComparing Summary
|
Summary
Rspack has switched to the npm trusted publishing, the OTP release scripts should be removed as they are no longer used.
Related links
Checklist