Skip to content

ci: add ruff lint and format checks - #1

Merged
dzmitry-duboyski merged 6 commits into
mainfrom
ci/ruff
Sep 20, 2026
Merged

dzmitry-duboyski merged 6 commits into
mainfrom
ci/ruff

Conversation

@dzmitry-duboyski

Copy link
Copy Markdown
Contributor

Add ruff to dev dependencies with a py39-targeted config, run ruff check --fix and ruff format across the codebase, and add a lint job to the tests workflow that gates the test matrix. Export __version__ in __all__ and add the Ruff badge to README.

dzmitry-duboyski and others added 6 commits September 20, 2026 19:46
Add ruff to dev dependencies with a py39-targeted config, run
`ruff check --fix` and `ruff format` across the codebase, and add a
lint job to the tests workflow that gates the test matrix. Export
`__version__` in `__all__` and add the Ruff badge to README.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Add mypy and types-requests to dev dependencies with a strict config,
run it in the lint job, and add the mypy badge to README.

Fix the eight no-any-return errors in both clients: validate that the
API response is a JSON object, coerce taskId/balance to int/float, and
check that a ready task carries a solution object. Behaviour is
unchanged for well-formed responses; malformed ones now raise
NetworkError instead of failing later.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
After a successful PyPI upload, create the matching GitHub Release with
auto-generated notes and the built sdist/wheel attached. Grant the
publish job contents:write for that. Add the GitHub Release badge to
README.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@dzmitry-duboyski
dzmitry-duboyski merged commit c7af29e into main Sep 20, 2026
6 checks passed
@dzmitry-duboyski
dzmitry-duboyski deleted the ci/ruff branch September 20, 2026 19:41
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.

1 participant