Skip to content

ci(nodejs): enable npm trusted publishing#6685

Merged
Xuanwo merged 1 commit intomainfrom
npm-trusted-publishing
Oct 17, 2025
Merged

ci(nodejs): enable npm trusted publishing#6685
Xuanwo merged 1 commit intomainfrom
npm-trusted-publishing

Conversation

@kingsword09
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #6648 #6484.

Rationale for this change

As Title.

What changes are included in this PR?

Are there any user-facing changes?

@kingsword09 kingsword09 requested a review from Xuanwo as a code owner October 17, 2025 07:10
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels Oct 17, 2025
@kingsword09
Copy link
Copy Markdown
Contributor Author

The last upgrade to napi 3.0 added new platform support, but the package has not been released yet. I don’t know if it will have any impact.

@kingsword09
Copy link
Copy Markdown
Contributor Author

After merging, can consider deleting NPM_TOKEN in the https://github.com/apache/opendal/settings/secrets/actions configuration.

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 17, 2025
@Xuanwo Xuanwo merged commit f9305fc into main Oct 17, 2025
24 checks passed
@Xuanwo Xuanwo deleted the npm-trusted-publishing branch October 17, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NPM Token Changes and Security Hardening

2 participants