Add tests, pre-commit hooks, CI, and refactor for pylint 10/10 #1

Merged

maks merged 4 commits from feat/tests-linting-ci into master 2026-03-31 19:35:48 +02:00

Conversation 0 Commits 4 Files changed 15 +2402 -664

maks commented 2026-03-31 17:18:54 +02:00

Owner

Copy link

• Add 141 tests across parser, db, pricing, and scraper modules
• Add pre-commit hooks (black, isort, pylint) excluding tests/
• Add Forgejo Actions CI with format, lint, and test jobs
• Refactor all source modules to pass pylint with no suppressions:
  • Extract helper functions to reduce complexity (locals/branches/args)
  • Narrow exception handling from broad Exception to specific types
  • Use context dicts and dispatch tables to reduce argument counts
  • Add docstrings to all public modules, classes, and methods
  • Move imports to top level, remove unused imports
• Configure isort with black-compatible profile

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

- Add 141 tests across parser, db, pricing, and scraper modules - Add pre-commit hooks (black, isort, pylint) excluding tests/ - Add Forgejo Actions CI with format, lint, and test jobs - Refactor all source modules to pass pylint with no suppressions: - Extract helper functions to reduce complexity (locals/branches/args) - Narrow exception handling from broad Exception to specific types - Use context dicts and dispatch tables to reduce argument counts - Add docstrings to all public modules, classes, and methods - Move imports to top level, remove unused imports - Configure isort with black-compatible profile Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks added 1 commit 2026-03-31 17:18:54 +02:00

Add tests, pre-commit hooks, CI, and refactor for pylint 10/10 ... d4a98113c8

- Add 141 tests across parser, db, pricing, and scraper modules
- Add pre-commit hooks (black, isort, pylint) excluding tests/
- Add Forgejo Actions CI with format, lint, and test jobs
- Refactor all source modules to pass pylint with no suppressions:
  - Extract helper functions to reduce complexity (locals/branches/args)
  - Narrow exception handling from broad Exception to specific types
  - Use context dicts and dispatch tables to reduce argument counts
  - Add docstrings to all public modules, classes, and methods
  - Move imports to top level, remove unused imports
- Configure isort with black-compatible profile

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks added 1 commit 2026-03-31 17:32:42 +02:00

Trigger CI ... cf8191d385

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks added 1 commit 2026-03-31 17:33:07 +02:00

Fix CI triggers: add master to branch list ... 02df9480f6

The default branch is master, not main. Without master in the
trigger list, PRs targeting master won't run CI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks force-pushed feat/tests-linting-ci from 02df9480f6 to b302ae23a8 2026-03-31 17:38:50 +02:00 Compare

maks added 1 commit 2026-03-31 19:26:42 +02:00

Use native runner with system Python instead of setup-python ... 8b4e35e4af

The self-hosted runner now has Python 3.13 via pyenv. Drop
actions/setup-python and use runs-on: native to run directly
on the host.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks force-pushed feat/tests-linting-ci from 8b4e35e4af to 96b60264be 2026-03-31 19:27:12 +02:00 Compare

maks added 1 commit 2026-03-31 19:28:37 +02:00

Replace actions/checkout with git clone for native runner ... 20491dcc27

The native runner has no Node.js, which actions/checkout@v4
requires. Use git clone directly instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks added 1 commit 2026-03-31 19:29:56 +02:00

Pin black target-version to py313 ... a0f3d6c730

Black on Python 3.13 cannot safety-check code formatted for 3.14.
Set target-version in pyproject.toml and reformat affected test files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

maks merged commit 938bf12763 into master 2026-03-31 19:35:48 +02:00

maks referenced this pull request from a commit 2026-03-31 19:35:49 +02:00

Merge pull request 'Add tests, pre-commit hooks, CI, and refactor for pylint 10/10' (#1) from feat/tests-linting-ci into master

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

maks/avtonet-monitor!1

No description provided.