Skip to content

Conversation

@sreesushmavad
Copy link

I updated the .pylintrc file to make our linting output more useful and less noisy.

  • Disabled common docstring warnings (C0114, C0115, C0116) and R0903 since they were cluttering the output.
  • Set max-line-length=100 for readability.
  • Ignored the tests folder during linting.
  • Added pydantic to the extension whitelist.

Resolved merge conflicts so that local changes and the upstream config are now in sync.

Result: Linting is now more consistent across environments, and the pylint score became more meaningful (improved from ~8.0 → ~9.26 ).

@sreesushmavad sreesushmavad changed the title Simplify and sync .pylintrc config for cleaner linting Simplify and sync .pylintrc config for cleaner linting (Fixes #1456) Sep 23, 2025
@bashir2 bashir2 requested a review from ndegwamartin October 15, 2025 15:46
@ndegwamartin
Copy link
Collaborator

@sreesushmavad again thank you for this. As mentioned on the Github issue here #1456 (comment) we already had a Pylint configuration added on this PR which has since been merged #1447. The configuration was updated with some sensible defaults some of which are in line with your recommendations on the PR here, otherwise the configuration borrows mostly from the Google Python Style Guide.

For Pydantic we haven't enabled that at the moment but we have added static type checking via Pyright here #1520

Given the above I'm going to close the PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants