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
Add new rule no-deprecated-router-transition-methods
#1715
Add new rule no-deprecated-router-transition-methods
#1715
Conversation
no-deprecated-router-transition-methods
I've updated the tests to have some examples of multiple classes (and fixes), updated the readme as requested, and have made changes to the error message to use |
LGTM - but looks like some tests / lint errors are still failing |
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.
Great progress. Left comments mostly about tests.
75faccf
to
746dec3
Compare
746dec3
to
211f50e
Compare
Updated this PR with requested feedback and learnings from the Added nested class tests, class expression tests, and accounted for legacy |
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.
Thank you!
no-deprecated-router-transition-methods
no-deprecated-router-transition-methods
Similar to #1714 this implements a lint rule to detect and fix errors associated with the
routing.transition-methods
deprecation.This closes #1074