Add setting the registry in the npmrc#14353
Conversation
|
The following pipelines have been queued for testing: |
There was a problem hiding this comment.
Pull request overview
This PR updates the shared npm publish pipeline to explicitly reset the project npm registry back to https://registry.npmjs.org/ during npmjs releases, addressing cases where a repo .npmrc might point at a different registry.
Changes:
- Added a new pipeline step template to set (and log) the project npm registry via
npm config. - Replaced the inline “delete repo .npmrc” step in the npm publish job with the new reset-registry template.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| eng/common/pipelines/templates/steps/reset-npmrc.yml | New step template that sets the project npm registry to the provided value. |
| eng/common/pipelines/templates/jobs/npm-publish.yml | Switches npmjs.org publish path to use the new reset-registry template instead of deleting .npmrc. |
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14353 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14353 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
Packages impacted by this PR
Issues associated with this PR
Azure/azure-sdk-for-js#37076
Describe the problem that is addressed by this PR