Skip to content

Conversation

@tschm
Copy link
Contributor

@tschm tschm commented Nov 10, 2025

Add this stage we do not yet install uv/uvx. We assume the user is familiar with uv.

@tschm
Copy link
Contributor Author

tschm commented Nov 10, 2025

@fkiraly This is the move to uv. I have updated the pyproject.toml file for it, removed poetry.lock and created a uv.lock file with uv sync --all-extras Again, I have deactivated the linters (causing mild problems)

@tschm
Copy link
Contributor Author

tschm commented Nov 10, 2025

@fkiraly this could be merged

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

Great! I fully support a move of CI to uv.

May I ask you though to only do one things in a PR at a time? This PR also includes changes to linting.

Also, it seems to be missing a change from pip install to uv pip install as one would expect in a PR that moves to uv?

@fkiraly fkiraly added the packaging A problem with installation, dependencies, versioning label Nov 10, 2025
@fkiraly fkiraly changed the title move to uv (assume you have a global uv/uvx installed) [MNT] move CI to uv (assume you have a global uv/uvx installed) Nov 10, 2025
@tschm
Copy link
Contributor Author

tschm commented Nov 11, 2025

Fair point to use uv pip install.

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

I think this PR is now superseded, except for the linting?

I moved the CI to uv separately.

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

There are actual changes to pyproject.toml, some revert earlier changes, such as adding python 3.13 and 3.14 compatibility, isolation of soft dependencies, etc.

Can you please revert these?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging A problem with installation, dependencies, versioning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants