Skip to content

build: suppress pnpm output in local-dev.sh#3555

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:silent-pnpm
Mar 17, 2026
Merged

build: suppress pnpm output in local-dev.sh#3555
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:silent-pnpm

Conversation

@alan-agius4
Copy link
Copy Markdown
Contributor

Adds the silent flag (-s) to the pnpm command in the local-dev.sh script to reduce noise when building the ng-dev package.

Adds the silent flag (`-s`) to the `pnpm` command in the `local-dev.sh`
script to reduce noise when building the `ng-dev` package.
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Mar 17, 2026
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Mar 17, 2026
Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds the silent flag (-s) to the pnpm command in the local-dev.sh script. This change aims to reduce the output noise when building the ng-dev package, which is a good developer experience improvement. The implementation is correct and achieves the intended goal. I have no further feedback.

@alan-agius4 alan-agius4 merged commit 1a4e12c into angular:main Mar 17, 2026
13 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor Author

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4 alan-agius4 deleted the silent-pnpm branch March 17, 2026 13:46
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants