Skip to content

fix(language-service): Add plugin option to force strictTemplates#41062

Closed
kyliau wants to merge 1 commit intoangular:masterfrom
kyliau:override-strict-templates
Closed

fix(language-service): Add plugin option to force strictTemplates#41062
kyliau wants to merge 1 commit intoangular:masterfrom
kyliau:override-strict-templates

Conversation

@kyliau
Copy link
Contributor

@kyliau kyliau commented Mar 3, 2021

This commit adds a new configuration option, forceStrictTemplates to the
language service plugin to allow users to force enable strictTemplates.

This is needed so that the Angular extension can be used inside Google without
changing the underlying compiler options in the ng_module build rule.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@kyliau kyliau added area: language-service Issues related to Angular's VS Code language service target: patch This PR is targeted for the next patch release labels Mar 3, 2021
@kyliau kyliau requested review from alxhub, atscott and zarend March 3, 2021 00:31
@ngbot ngbot bot modified the milestone: Backlog Mar 3, 2021
@google-cla google-cla bot added the cla: yes label Mar 3, 2021
@kyliau kyliau added design complexity: major target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release design complexity: major target: major This PR is targeted for the next major release labels Mar 3, 2021
@kyliau kyliau force-pushed the override-strict-templates branch from 2152479 to b221adf Compare March 3, 2021 01:06
@kyliau kyliau changed the title fix(language-service): Add plugin option to override strictTemplates fix(language-service): Add plugin option to force strictTemplates Mar 3, 2021
@kyliau kyliau added the target: major This PR is targeted for the next major release label Mar 3, 2021
@kyliau kyliau force-pushed the override-strict-templates branch from b221adf to 2b5eee9 Compare March 3, 2021 01:21
@kyliau kyliau changed the title fix(language-service): Add plugin option to force strictTemplates feat(language-service): Add plugin option to force strictTemplates Mar 3, 2021
@kyliau kyliau force-pushed the override-strict-templates branch from 2b5eee9 to d8551f4 Compare March 3, 2021 03:29
This commit adds a new configuration option, `forceStrictTemplates` to the
language service plugin to allow users to force enable `strictTemplates`.

This is needed so that the Angular extension can be used inside Google without
changing the underlying compiler options in the `ng_module` build rule.
@kyliau kyliau force-pushed the override-strict-templates branch from d8551f4 to f4f7e64 Compare March 3, 2021 03:34
@kyliau kyliau changed the title feat(language-service): Add plugin option to force strictTemplates fix(language-service): Add plugin option to force strictTemplates Mar 3, 2021
@kyliau kyliau added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker and removed target: major This PR is targeted for the next major release labels Mar 3, 2021
@zarend zarend removed the target: patch This PR is targeted for the next patch release label Mar 3, 2021
This was referenced Mar 15, 2021
@angular-automatic-lock-bot
Copy link

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 Apr 3, 2021
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: language-service Issues related to Angular's VS Code language service cla: yes target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants