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.

Sign up for GitHub

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

Jump to bottom

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

Open
8 tasks
l0lawrence opened this issue Apr 8, 2022 · 2 comments
Open
8 tasks

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

l0lawrence opened this issue Apr 8, 2022 · 2 comments
Assignees
@l0lawrence
Labels
good first issue This issue tracks work that may be a good starting point for a first-time contributor pylint priority 1 pylint Python

Comments

@l0lawrence
Copy link
Member

l0lawrence commented Apr 8, 2022 •
edited
Loading

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))
@l0lawrence l0lawrence self-assigned this Apr 8, 2022
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Apr 26, 2022
@l0lawrence l0lawrence transferred this issue from Azure/azure-sdk-for-python Apr 26, 2022
@l0lawrence l0lawrence added Python pylint priority 1 and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Apr 26, 2022
@l0lawrence l0lawrence added the good first issue This issue tracks work that may be a good starting point for a first-time contributor label Jul 23, 2024
@l0lawrence l0lawrence moved this from 📋 Backlog to 🆕 New in Pylint Feature Rollout (python-sdk) Aug 1, 2024
@MJoshuaB
Copy link
Contributor

I am currently reviewing this Issue.

@MJoshuaB
Copy link
Contributor

MJoshuaB commented Oct 2, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@l0lawrence l0lawrence

Labels
good first issue This issue tracks work that may be a good starting point for a first-time contributor pylint priority 1 pylint Python
Projects
Pylint Feature Rollout (python-sdk)
Status: 🆕 ToDo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MJoshuaB @l0lawrence