Skip to content

Conversation

@nzakas
Copy link
Member

@nzakas nzakas commented Oct 9, 2025

Prerequisites checklist

What is the purpose of this pull request?

Switch to Trusted Publishing.

What changes did you make? (Give an overview)

  • Ensured that changes to release-please.yml require an approval from TSC
  • Ensured that npm is always the latest version to ensure availability of trusted publishing
  • Removed NPM_TOKEN from workflow
  • Set the npm permissions for this package to disallow token publishing.

Related Issues

refs eslint/rewrite#280

Is there anything you'd like reviewers to focus on?

@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 9, 2025
Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 10, 2025
@lumirlumir lumirlumir merged commit f52c95c into main Oct 10, 2025
22 checks passed
@lumirlumir lumirlumir deleted the trusted-publishing branch October 10, 2025 04:00
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion build

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants