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
- Remove (or clear) the
ignore-patterns key under [tool.pylint.main] in pyproject.toml.
- Run
pylint tests/ to enumerate all existing complaints.
- Fix or suppress each complaint with inline
# pylint: disable=... comments or project-level config adjustments where suppression is genuinely warranted.
- Ensure the CI pylint job passes with test files included.
References
/cc @galshubeli
Overview
Pylint currently skips
tests/viaignore-patternsinpyproject.toml: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-patternsfrom[tool.pylint.main]so pylint covers all Python files, including test files, as per the project's coding guidelines.Work required
ignore-patternskey under[tool.pylint.main]inpyproject.toml.pylint tests/to enumerate all existing complaints.# pylint: disable=...comments or project-level config adjustments where suppression is genuinely warranted.References
/cc @galshubeli