Skip to content

Conversation

@Thijss
Copy link
Member

@Thijss Thijss commented Sep 12, 2025

This PR migrates the python package management to uv (https://docs.astral.sh/uv/) and adds poethepoet to simplify/streamline CI/CD checks.

  • A uv.lock file is added
  • poethepoet is now used to run CI/CD tasks
    • tasks are added to pyproject.toml
  • dev-dependencies are moved from optional to group dependencies (following https://docs.astral.sh/uv/concepts/projects/dependencies/)
  • Github Actions workflows are simplified and updated to work with uv
    • nigthly action now runs with a uv update command.
  • Dependencies are updated since these weren't explicitly locked before.
  • Python file changes:
    • Some style/lint issues have been addressed, to accommodate issues raised by new versions of mypy/ruff/etc.
  • pre-commit now only runs poe all --check
    • small downside: it now always runs on all files, instead of just staged files.

ToDo

  • Possibly mention uv in the README.
  • use uv --frozen where possible
  • Create test release to check whether extras (e.g. visualizer) are still working correctly

Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
…ly.github.com>

I, Thijs Baaijen <[email protected]>, hereby add my Signed-off-by to this commit: ce06ef0
I, Thijs Baaijen <[email protected]>, hereby add my Signed-off-by to this commit: afbc4c6

Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
@Thijss Thijss changed the title Switch to uv Switch to uv and poethepoet Sep 12, 2025
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
Signed-off-by: Thijs Baaijen <[email protected]>
@sonarqubecloud
Copy link

@Thijss Thijss marked this pull request as ready for review September 12, 2025 11:06
Copy link
Member

@jaapschoutenalliander jaapschoutenalliander left a comment

Choose a reason for hiding this comment

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

Nice work, let's try how it it works in this setup and if any addition/change is needed we can iterate

@jaapschoutenalliander jaapschoutenalliander merged commit 45a63b1 into main Oct 1, 2025
23 checks passed
@jaapschoutenalliander jaapschoutenalliander deleted the switch-to-uv branch October 1, 2025 13:20
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.

3 participants