Skip to content

docs(@angular/build): correct outputFile and port option descriptions - #34404

Open
MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:docs/build-option-descriptions
Open

MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:docs/build-option-descriptions

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

The unit-test outputFile description stated that the path applies only to the first reporter, but it is passed to Vitest as is and used by every configured reporter that writes to a file. The dev-server port description did not mention that the PORT environment variable takes precedence over the option.

…ions

The unit-test `outputFile` description stated that the path applies only to the first
reporter, but it is passed to Vitest as is and used by every configured reporter that
writes to a file. The dev-server `port` description did not mention that the `PORT`
environment variable takes precedence over the option.

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

Copy link
Copy Markdown

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 updates the schema descriptions for the Angular build tools. Specifically, it clarifies that the 'PORT' environment variable takes precedence over the 'port' option in the dev-server builder, and updates the 'outputFile' description in the unit-test builder to explain how the path is utilized by multiple reporters. I have no feedback to provide.

@angular-robot angular-robot Bot added area: docs Related to the documentation area: @angular/build labels Oct 19, 2026

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks

@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Oct 19, 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: @angular/build area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants