Skip to content

build: add explanation comment to confirmModulesPurge setting#68852

Open
alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:pnpm-purge
Open

build: add explanation comment to confirmModulesPurge setting#68852
alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:pnpm-purge

Conversation

@alan-agius4
Copy link
Copy Markdown
Contributor

Add a detailed comment to the confirmModulesPurge: false setting in pnpm-workspace.yaml

Add a detailed comment to the `confirmModulesPurge: false` setting in
`pnpm-workspace.yaml`. This comment explains that this setting avoids
interactive prompts when `node_modules` needs to be purged and
recreated. This prevents package installation commands from hanging or
failing in non-interactive contexts (such as CI/CD pipelines, container
builds, or automated editor/agent environments) where stdin is
not a TTY.
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels May 21, 2026
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label May 21, 2026
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels May 21, 2026
@ngbot ngbot Bot added this to the Backlog milestone May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants