[pylint] Address Commented out Pylint Custom Plugin Checkers - Githubissues

Azure / azure-sdk-tools

Tools repository leveraged by the Azure SDK team.

MIT License

114 stars 177 forks source link

[pylint] Address Commented out Pylint Custom Plugin Checkers #3228

Open l0lawrence opened 2 years ago

l0lawrence commented 2 years ago

Investigate the following rules and why they are disabled (false positives?) and how to improve the quality or if they should be removed completely

[ ] - [ ] # disabled by default, linter.register_checker(CheckDocstringParameters(linter))
- [ ] # linter.register_checker(CheckForPolicyUse(linter))
- [ ] # linter.register_checker(ClientHasApprovedMethodNamePrefix(linter))
- [ ] # linter.register_checker(ClientMethodsHaveTracingDecorators(linter))
- [ ] # linter.register_checker(ClientDocstringUsesLiteralIncludeForCodeExample(linter))
- [ ] # linter.register_checker(ClientListMethodsUseCorePaging(linter))
- [ ] # linter.register_checker(ClientLROMethodsUseCorePolling(linter))
- [ ] # linter.register_checker(ClientLROMethodsUseCorrectNaming(linter))

MJoshuaB commented 2 months ago

I am currently reviewing this Issue.

MJoshuaB commented 1 month ago

I have submitted PR #9081 addressing ClientHasApprovedMethodNamePrefix I am not working on the other checkers listed at this time.