Skip to content

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

@coderabbitai

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

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions