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
Linter for global function names #425
Linter for global function names #425
Conversation
changes made! |
Looks good! |
Hmm.. maybe we should generate test cases for each function? If we don't test them then the test coverage will suffer as there will be so many lines that are not tested. |
7b1411d
to
1759f36
Compare
I had some spare time and put in a whole bunch more test cases. There's more that need to be put in, but this is a lot more than we had earlier. |
I think that this is good now. I need to look through the changes once more before merging. |
Issue: #397
This creates a linter for global function names.
Couple things:
I didn't know how comprehensive to make the docs + test suite. Putting in examples for every rule seemed excessively verbose.
I purposefully didn't put in auto-fix functionality, since we're encouraging people to use the migrator. Does that seem safe?