Skip to content

feat(compiler-cli): 'strictStandalone' flag enforces standalone - #57935

Closed
alxhub wants to merge 1 commit into
angular:mainfrom
alxhub:strict-standalone
Closed

alxhub wants to merge 1 commit into
angular:mainfrom
alxhub:strict-standalone

Conversation

@alxhub

@alxhub alxhub commented Sep 23, 2024

Copy link
Copy Markdown
Member

Add the strictStandalone flag to angularCompilerOptions. When set to true, the compiler will require that all declarations of components, directive, and pipes be standalone. When standalone: false is provided, an error is raised.

Note that until the default value of the standalone flag is flipped, this does not catch the case where a declaration does not specify a value for standalone.

The default value of the strictStandalone flag is false.

@angular-robot angular-robot Bot added the detected: feature PR contains a feature commit label Sep 23, 2024
Add the `strictStandalone` flag to `angularCompilerOptions`. When set to
true, the compiler will require that all declarations of components,
directive, and pipes be standalone. When `standalone: false` is provided,
an error is raised.

Note that until the default value of the standalone flag is flipped, this
does not catch the case where a declaration does not specify a value for
`standalone`.

The default value of the `strictStandalone` flag is `false`.
@alxhub
alxhub marked this pull request as ready for review September 24, 2024 23:22
@pullapprove
pullapprove Bot requested a review from atscott September 25, 2024 00:24

@AndrewKushnir AndrewKushnir left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed-for: public-api

@pkozlowski-opensource pkozlowski-opensource added the area: compiler Issues related to `ngc`, Angular's template compiler label Sep 25, 2024
@ngbot ngbot Bot added this to the Backlog milestone Sep 25, 2024
@alxhub alxhub added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Sep 26, 2024
@alxhub

alxhub commented Sep 26, 2024

Copy link
Copy Markdown
Member Author

This PR was merged into the repository by commit d9687f4.

The changes were merged into the following branches: main

@alxhub alxhub closed this in d9687f4 Sep 26, 2024
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Oct 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: compiler Issues related to `ngc`, Angular's template compiler detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants