Skip to content

refactor(@schematics/angular): remove explicit strict true in tsconfig template for TS6 compatibility#32993

Open
alan-agius4 wants to merge 1 commit intoangular:mainfrom
alan-agius4:strict-ts
Open

refactor(@schematics/angular): remove explicit strict true in tsconfig template for TS6 compatibility#32993
alan-agius4 wants to merge 1 commit intoangular:mainfrom
alan-agius4:strict-ts

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

TypeScript 6 enables strict mode by default. This change updates the workspace schematic's tsconfig.json template to omit "strict": true when strict mode is desired (relying on the default), and explicitly sets "strict": false when it is disabled.

…g template for TS6 compatibility

TypeScript 6 enables strict mode by default. This change updates the workspace schematic's `tsconfig.json` template to omit `"strict": true` when strict mode is desired (relying on the default), and explicitly sets `"strict": false` when it is disabled.
@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 Apr 13, 2026
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 modifies the tsconfig.json template to remove the explicit "strict": true setting when strict mode is enabled, while adding an explicit "strict": false when it is disabled. Feedback suggests that removing the explicit strict setting relies on TypeScript 6.0 defaults and will silently disable strict mode for users on older versions. It is recommended to keep the "strict": true setting explicit to ensure consistent type safety across all supported TypeScript versions and to update the corresponding tests to reflect this requirement.

@alan-agius4 alan-agius4 requested a review from clydin April 13, 2026 15:03
@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 Apr 13, 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: @schematics/angular 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