Backport forceauth removal 20.3.x - #33339
Conversation
clydin
added
the
target: lts
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request removes support and testing for unscoped registry authentication across the Angular CLI package metadata utilities and legacy CLI end-to-end tests. Specifically, it removes the handling of auth options in normalizeOptions and cleans up associated test cases that previously verified unscoped authentication. Feedback suggests further simplifying the createNpmConfigForAuthentication utility by completely removing the now-unused scopedAuthentication parameter and its corresponding dead code branch.
09a2c09 to
26cb904
Compare
Remove the non-standard forceAuth option and custom parsing for unscoped registry credentials (token, username, password, auth) in package-metadata.ts. Since the npm CLI does not support unscoped credentials and ignores them by default, this aligning removes the unnecessary parsing complexity. Unscoped credentials will now behave identically to any other standard configuration property, falling through to default configuration parsing.
26cb904 to
b076a42
Compare
clydin
added
the
action: review
alan-agius4
added
action: merge
|
This pull request has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
2 participants
No description provided.