Skip to content

Use default npmrc userprofile#11045

Open
chidozieononiwu wants to merge 3 commits into
mainfrom
net-iso
Open

Use default npmrc userprofile#11045
chidozieononiwu wants to merge 3 commits into
mainfrom
net-iso

Conversation

@chidozieononiwu

Copy link
Copy Markdown
Member

This pull request standardizes the creation and authentication of .npmrc files across multiple pipelines by introducing a new reusable template in eng/common/pipelines/templates/steps/create-authenticated-npmrc.yml. It replaces older, package-specific implementations and updates various pipeline YAML files to use this new template, ensuring consistency and maintainability in how authenticated npm registry access is handled.

Centralization and Standardization of npm Authentication:

  • Added a new, parameterized template create-authenticated-npmrc.yml in eng/common/pipelines/templates/steps/ to create and authenticate .npmrc files for npm registry access. This template supports customizable parameters and sets up authentication using the npmAuthenticate@0 task.
  • Updated multiple pipeline files (including ci.yml, build-step.yml, test-step.yml, install.yml, and others) to use the new common template for npm authentication, replacing direct script steps and previous template usages. [1] [2] [3] [4] [5] [6] [7]

Cleanup and Removal of Redundant Code:

  • Removed the old create-authenticated-npmrc.yml template from eng/emitters/pipelines/templates/steps/, which previously handled .npmrc creation and authentication in a less flexible manner.
  • Updated emitter pipeline stages to reference the new centralized template instead of the deleted emitter-specific version.

These changes improve maintainability by reducing duplication, making it easier to update npm authentication logic in one place, and ensuring all pipelines use the same approach for authenticated npm registry access.

@pkg-pr-new

pkg-pr-new Bot commented Jun 22, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-csharp@11045

commit: 44eef55

@github-actions

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp eng

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant