-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
build: enable coercion-types rule and fix leftover cases #17609
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left out all of these since they are only for examples and my thought was that we don't want to show these in the docs. They might be rather confusing since the syntax is quite new.
LGTM if we want to go with that though.
f41cdfd
to
1ce6b3b
Compare
I did it for the stepper example, because it shows how to extend a stepper so anybody doing it will have to do it as well. I reworked the rest since they are either internal or didn't have to coerce in the first place. |
1ce6b3b
to
a6c6f5a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Doing the custom stepper since it is advanced, sounds good to me 👍
...mples/cdk/stepper/cdk-custom-stepper-without-form/cdk-custom-stepper-without-form-example.ts
Show resolved
Hide resolved
Follow-up from angular#17528 that: * Turns on the `coercion-types` rule. * Fixes the rule not detecting abstract directives properly. * Fixes the rule incorrectly flagging coercion functions used inside of callbacks which are inside of setters. * Either fixes or works around a final set of failures.
a6c6f5a
to
ede078b
Compare
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Follow-up from #17528 that:
coercion-types
rule.