MetaMask / action-npm-publish

GitHub Action to publish to NPM
MIT License
8 stars 6 forks source link

Rename `YARN_NPM_TAG` to `PUBLISH_NPM_TAG` #32

Closed Mrtenz closed 1 year ago

Mrtenz commented 1 year ago

Description

This pull request renames the YARN_NPM_TAG environment variable to PUBLISH_NPM_TAG. The reason for doing so is that environment variables starting with YARN_ are reserved for Yarn configuration options. Hence, if we use YARN_NPM_TAG as an environment variable, it will cause Yarn to throw an error. On the other hand, PUBLISH_NPM_TAG is not a reserved environment variable name, which makes it a preferred choice.

Changes

  1. Rename YARN_NPM_TAG to PUBLISH_NPM_TAG.