Skip to content

ci(W-24555083): remove npm token save from promote workflow - #2982

Merged
k80bowman merged 1 commit into
mainfrom
k80/remove-npm-token
Oct 21, 2026
Merged

k80bowman merged 1 commit into
mainfrom
k80/remove-npm-token

Conversation

@k80bowman

Copy link
Copy Markdown
Contributor

What does this PR do?

Describe the feature with use cases or bug details. Interesting/complex design choices or weird workarounds should be documented in the code but can be included here instead if there is a valid reason it shouldn't be in the code.

Removes the script that saves the NPM token to a .npmrc file from the promote workflow in order to enable Trusted Publishing.

Acceptance Criteria

How do you know this change is successful? What is the scope of this change? What tests test the criteria (or why no tests)?

The Acceptance Criteria can be copied from the work item if they exist there but it is useful to have on the PR when reviewing the code.

I will validate that this change is working next Wednesday when the promotion workflows run.

Testing Notes

Anything additional to note about testing this PR. How did you test it? Special setup? Additional manual test cases? Things not tested?

Checklist

  • This change has been approved by the CLI Review Board or approval isn't required. Anything that adds/changes/deletes commands, flags, output, or json output has to be reviewed.
  • Does this follow the deprecation policy?

What issues does this PR fix or reference?

@W-24555083@

@github-actions

Copy link
Copy Markdown

This PR lacks any commits of the 'fix' or 'feat' type, and therefore will not trigger a release. To silence all further warnings, react to this warning comment (or any other) with the 👀 emoji.

NOTE: If your repo uses squash commits, make sure to add the appropriate conventional commit prefix (e.g., 'fix:', 'feat:', 'chore:', etc) in the squash commit message.

@k80bowman
k80bowman merged commit 55d91a5 into main Oct 21, 2026
37 checks passed
@k80bowman
k80bowman deleted the k80/remove-npm-token branch October 21, 2026 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants