Skip to content

chore: enable pylint for test files (test_*.py, conftest.py) #560

Description

@coderabbitai

Overview

Pylint currently skips tests/ via ignore-patterns in pyproject.toml:

[tool.pylint.main]
ignore-patterns = ["test_.*\\.py", "conftest\.py"]

This was intentionally deferred in PR #544 (SDK packaging refactor) because removing the exclusion surfaces a backlog of pre-existing pylint complaints across tests/ (e2e, conftest fixtures, integration tests) that are unrelated to that PR's scope.

Goal

Remove ignore-patterns from [tool.pylint.main] so pylint covers all Python files, including test files, as per the project's coding guidelines.

Work required

  1. Remove (or clear) the ignore-patterns key under [tool.pylint.main] in pyproject.toml.
  2. Run pylint tests/ to enumerate all existing complaints.
  3. Fix or suppress each complaint with inline # pylint: disable=... comments or project-level config adjustments where suppression is genuinely warranted.
  4. Ensure the CI pylint job passes with test files included.

References

/cc @galshubeli

Activity

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

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions