Skip to content

**fix** restore CI tox install and drop Poetry from tox runs - #58

Merged
FirefighterBlu3 merged 1 commit into
mainfrom
fix/ci-tox-requirements
Jul 26, 2026
Merged

**fix** restore CI tox install and drop Poetry from tox runs#58
FirefighterBlu3 merged 1 commit into
mainfrom
fix/ci-tox-requirements

Conversation

@FirefighterBlu3

Copy link
Copy Markdown
Owner

Summary

  • Regenerates .github/requirements-ci.txt (it was committed empty, so CI installed nothing and tox was missing)
  • Runs python -m tox and verifies install in main.yml
  • Makes tox invoke pytest/mypy/pylint directly (no Poetry binary required in CI)
  • Fixes update-ci-requirements pip-tools/pip compatibility and refuses an empty compile output

Test plan

Made with Cursor

requirements-ci.txt was committed empty so pip installed nothing; regenerate
hashes, run tox via python -m, and execute tests without a Poetry binary.

Co-authored-by: Cursor <cursoragent@cursor.com>
@FirefighterBlu3 FirefighterBlu3 self-assigned this Jul 26, 2026

@FirefighterBlu3 FirefighterBlu3 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@FirefighterBlu3
FirefighterBlu3 merged commit 6428526 into main Jul 26, 2026
6 checks passed
@FirefighterBlu3
FirefighterBlu3 deleted the fix/ci-tox-requirements branch July 26, 2026 03:52
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