Skip to content

docs(@angular/build): correct the unit-test browsers option description - #34589

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
MeAkib:docs/unit-test-browsers-description
Oct 30, 2026
Merged

alan-agius4 merged 1 commit into
angular:mainfrom
MeAkib:docs/unit-test-browsers-description

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

The description states that tests run with jsdom when no browsers are set and gives 'ChromeHeadless' as the only example. The Vitest runner prefers happy-dom when it is installed, the Karma runner uses the browsers of its configuration, and the Vitest Playwright provider rejects 'chrome', so the example fails with the most common provider. Describe both runners and give an example for each.

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

@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 description of the browsers property in the unit-test builder schema to clarify the default behaviors of the Vitest and Karma runners. The feedback suggests formatting technical terms and configuration values with backticks to maintain consistency with other schema descriptions.

@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.

Please squash the two commits (or update the fixup! commit message to include the backticks around `browsers`) so that the commit message lint check passes.

…tion

The description states that tests run with jsdom when no browsers are set and
gives 'ChromeHeadless' as the only example. The Vitest runner prefers happy-dom
when it is installed, the Karma runner uses the browsers of its configuration,
and the Vitest Playwright provider rejects 'chrome', so the example fails with
the most common provider. Describe both runners and give an example for each.
@MeAkib
MeAkib force-pushed the docs/unit-test-browsers-description branch from 3ac3a71 to db12eef Compare October 29, 2026 18:42
@MeAkib

MeAkib commented Oct 29, 2026

Copy link
Copy Markdown
Contributor Author

Please squash the two commits (or update the fixup! commit message to include the backticks around `browsers`) so that the commit message lint check passes.

Done

@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 29, 2026
@alan-agius4
alan-agius4 merged commit 2ebb5e1 into angular:main Oct 30, 2026
41 checks passed
@alan-agius4

Copy link
Copy Markdown
Collaborator

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

@MeAkib
MeAkib deleted the docs/unit-test-browsers-description branch October 30, 2026 11:42
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