Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Light up typescript check: strictDomLocalRefTypes #4028

Merged
merged 1 commit into from
Jul 6, 2020

Conversation

msorens
Copy link
Contributor

@msorens msorens commented Jul 2, 2020

🔩 Description: What code changed, and why?

Follow-on to Angular 9 template type checking enablement, this throws the switch on one more check.
This was an easy win--no collateral to fix after turning it on!

⛓️ Related Resources

PR #3268 Turn on strict Angular checks
PR #3158 Light up an additional typescript template check

👍 Definition of Done

ng build reports no errors or warnings

👟 How to Build and Test the Change

NA

✅ Checklist

Copy link
Contributor

@srenatus srenatus left a comment

Choose a reason for hiding this comment

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

No idea what this checks, but fine ;)

Copy link
Contributor

@tarablack01 tarablack01 left a comment

Choose a reason for hiding this comment

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

This is a big one 😉
LGTM 👍🏼

@msorens msorens merged commit 5332bc5 into master Jul 6, 2020
@msorens msorens deleted the ms/angular-checking branch July 6, 2020 16:58
@susanev susanev added this to the Auth: Sprint 18 milestone Jul 6, 2020
@susanev susanev modified the milestones: Auth: Sprint 18, Auth: Sprint 17 Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants