Skip to content
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

use NODE_AUTH_TOKEN as environment variable name for the NPM_TOKEN secret #266

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yaacovCR
Copy link

@yaacovCR yaacovCR commented Feb 26, 2023

depends on #267 which was extracted from this PR

context: #262 (comment)

in a follow-up PR, one could envision a persist: false option that removes these files if the action actually created them so as to perform "clean up" similar to the persist-credentials: false option for actions/checkout

@changeset-bot
Copy link

changeset-bot bot commented Feb 26, 2023

🦋 Changeset detected

Latest commit: bfe9b96

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@changesets/action Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@yaacovCR yaacovCR changed the title match NPM_TOKEN's environment variable name to official documentation write environment variable references to files instead of the values Feb 26, 2023
@yaacovCR yaacovCR changed the title write environment variable references to files instead of the values use NODE_AUTH_TOKEN as environment variable name for the NPM_TOKEN secret Feb 26, 2023
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.

1 participant