Skip to content

feat(@schematics/angular): rely on strict template default in generated workspaces#32824

Merged
alan-agius4 merged 2 commits intoangular:mainfrom
alan-agius4:strict-template
Mar 24, 2026
Merged

feat(@schematics/angular): rely on strict template default in generated workspaces#32824
alan-agius4 merged 2 commits intoangular:mainfrom
alan-agius4:strict-template

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 commented Mar 24, 2026

Since strict templates are now the default in the Angular compiler, this change removes the explicit "strictTemplates": true setting when creating a new strict workspace. We now only explicitly set "strictTemplates": false when the workspace is created with --strict=false to maintain consistent behavior for non-strict setups.

See: angular/angular#67802

@alan-agius4 alan-agius4 requested a review from JeanMeche March 24, 2026 09:02
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: major This PR is targeted for the next major release labels Mar 24, 2026
…ed workspaces

Since strict templates are now the default in the Angular compiler, this
change removes the explicit "strictTemplates": true setting when creating
a new strict workspace. We now only explicitly set "strictTemplates": false
when the workspace is created with --strict=false to maintain consistent
behavior for non-strict setups.

See: angular/angular#67802
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 correctly updates the tsconfig.json.template to reflect the Angular compiler's new default behavior for strictTemplates. It removes the explicit strictTemplates: true setting when a strict workspace is generated, and instead explicitly sets strictTemplates: false only when --strict=false is provided. The associated test cases in index_spec.ts have been updated to align with these changes. I've identified an opportunity to improve the test coverage for the strict: true scenario by adding assertions for other angularCompilerOptions that are explicitly set.

@alan-agius4 alan-agius4 requested a review from clydin March 24, 2026 13:13
@alan-agius4 alan-agius4 removed the request for review from JeanMeche March 24, 2026 14:04
@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 Mar 24, 2026
@alan-agius4 alan-agius4 merged commit f98cc82 into angular:main Mar 24, 2026
36 checks passed
@alan-agius4
Copy link
Copy Markdown
Collaborator Author

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

@alan-agius4 alan-agius4 deleted the strict-template branch March 24, 2026 14:05
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: @schematics/angular detected: feature PR contains a feature commit target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants