Skip to content

Conversation

@kown7
Copy link
Contributor

@kown7 kown7 commented Jul 30, 2025

Enable building and running with modern pyproject.toml compatible tooling. I hope this adresses the concerns raised in #39.

https://docs.astral.sh/uv

And some Claude AI slop removed.

@kown7 kown7 force-pushed the uv/enable branch 2 times, most recently from f396484 to 6e72aba Compare July 31, 2025 13:32
@kown7
Copy link
Contributor Author

kown7 commented Jul 31, 2025

Sonar error is not something I can fix from my side.

@florath
Copy link
Owner

florath commented Aug 2, 2025

Thanks for the hint to the rmtoo.log file. I removed it.

I will not merge the rest of the PR. My reasons:

  1. For me just updating something is not an end in itself. It does not add any functionality or provide any benefit to my knowledge.
  2. I personally do not know uv - so I need to run through a learning curve - where I do not see any reason for.
  3. I'd assume that the deployment script needs changes. Also here: it works perfectly at the moment.
  4. ... and this for more than 1200 new lines of code which needs maintenance.

@florath florath closed this Aug 2, 2025
@kown7
Copy link
Contributor Author

kown7 commented Aug 7, 2025

As and addendum for 1. and 4.: the reason why you would want that is a SBOM.

And yes, the lock file is autogenerated but guarantees stable dependencies. Something that setuptools alone can't ensure because it's recalculated everytime or unclear how and when.

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.

2 participants