Dependency setup

[flying-sheep]

Two questions here:

postmodern-python-ci/.github/workflows/ci.yml

Lines 26 to 27 in 1fbc53f

	uv export --resolution ${{ matrix.resolution }} > requirements.txt
	uv pip install --system --break-system-packages -r requirements.txt

• Shouldn’t we use a non-managed Python instead of using --break-system-packages? In Docker, plain --system works. We could just use astral-sh/setup-uv with python-version: ..., right?
• requirements.txt is not an ideal format, why not just use the uv lockfile?

[nathanjmcdougall]

It's definitely hacky.

• I'd like do away with setup-python and use a uv-managed Python installation (and setup-uv) instead of the system Python, but python-build-standalone has this issue:
  Python scripts do not work under valgrind, including codspeed astral-sh/python-build-standalone#508 which is incompatible with codspeed, which is a service I use.

• Those lines were a hack dating from back when uv still had an issue with lockfiles not handling dynamic project versions (Lock files don't mesh with dynamic package version info astral-sh/uv#7533). Now, I'd recommend using the lockfile. However, for some reason I tried making this change and it broke the codecov bot for PRs. I haven't had the time to investigate why. This was the offending PR:
  https://github.com/nathanjmcdougall/usethis-python/pull/334/files#diff-6c09a6667e4789699a21fff92cd1cc9542d647d2a2037d864bdc1de965c5d22b

• Regarding the need for --break-system-packages, I think you're right that's not necessary. When I made this 7 months ago I found that it didn't work without it (I don't recall why). I've removed it for one of my projects here and no obvious problems yet:
  Try removing --break-system-packages in ci.yml usethis-python/usethis-python#612
  All going well, I think it can be changed here in this repo too.

FWIW this repo is not something I'd given much thought to, it was a one-off proof of concept... forgive its grandiose-sounding name.

[flying-sheep]

Haha, yeah, I checked it out because it’s always interesting to see what nice things other have discovered.

[nathanjmcdougall]

I've updated the CI to use a more conventional approach, along with some other "spring cleaning" in line with what I see as best practice as of the uv v0.8.0 release.