Skip to content

docs: correct inlineStyle and Karma include option descriptions - #34542

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
MeAkib:docs/inline-style-and-karma-include
Oct 27, 2026
Merged

alan-agius4 merged 1 commit into
angular:mainfrom
MeAkib:docs/inline-style-and-karma-include

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

The ng new/application inlineStyle description claimed only CSS can be included inline, but the schematic sets inlineStyleLanguage for other style languages, so inline Sass and Less work. The Karma include description said directories pick up only .spec files, but test discovery matches both .spec and .test files with .ts or .tsx.

@angular-robot angular-robot Bot added the area: docs Related to the documentation label Oct 27, 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 schema descriptions in the Angular build and schematics packages. Specifically, it modifies the Karma builder schema to document support for both .spec and .test files, and simplifies the description of the inlineStyle option in the application schema. A review comment suggests updating the second bullet point in the Karma builder schema description to refer to a "matching test file" instead of a "matching spec file" to maintain consistency with the first bullet point.

Comment thread packages/angular/build/src/builders/karma/schema.json Outdated

@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

@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 27, 2026
The `ng new`/`application` `inlineStyle` description claimed only CSS can be included inline, but the
schematic sets `inlineStyleLanguage` for other style languages, so inline Sass and Less work. The
Karma `include` description said directories pick up only `.spec` files, but test discovery matches
both `.spec` and `.test` files with `.ts` or `.tsx`.
@alan-agius4
alan-agius4 force-pushed the docs/inline-style-and-karma-include branch from b0b36da to 2425175 Compare October 27, 2026 16:18
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: merge The PR is ready for merge by the caretaker labels Oct 27, 2026
@alan-agius4
alan-agius4 merged commit b1e814c into angular:main Oct 27, 2026
40 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/inline-style-and-karma-include branch October 28, 2026 02:45
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: 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