Skip to content

Conversation

HaoZeke
Copy link
Member

@HaoZeke HaoZeke commented Oct 8, 2025

Remains to be seen, but.. https://hynek.me/articles/turbo-charge-tox/

All times in minutes, and rounded.

Test Current Main
Architecture 2 3
Build 1.2 1.5
Documentation 3.4 4.5
Lint 21s 29s
Tests 6 7.3

Which is pretty good IMO for something which costs us nothing.


📚 Documentation preview 📚: https://metatrain--787.org.readthedocs.build/en/787/

@PicoCentauri
Copy link
Contributor

PicoCentauri commented Oct 8, 2025

Nice document! I think this only helps locally when you share an env, right?

Do we also have to set?

pass_env = SETUPTOOLS_SCM_PRETEND_VERSION

@HaoZeke HaoZeke marked this pull request as draft October 8, 2025 11:06
@HaoZeke HaoZeke force-pushed the fasterTox branch 5 times, most recently from 31a7db8 to 769b4f1 Compare October 8, 2025 16:43
@HaoZeke HaoZeke marked this pull request as ready for review October 8, 2025 17:03
@HaoZeke
Copy link
Member Author

HaoZeke commented Oct 8, 2025

Nice document! I think this only helps locally when you share an env, right?

Do we also have to set?

pass_env = SETUPTOOLS_SCM_PRETEND_VERSION

Yup or now that there's tox-uv with caching, it should still help. We don't need the environment variables since we thankfully don't set them.

@HaoZeke HaoZeke changed the title chore(tox): maybe faster chore(tox): faster with uv and caches Oct 8, 2025
@RMeli
Copy link
Member

RMeli commented Oct 8, 2025

Can you please update the CSCS CI too? Otherwise I'll take care of it.

@PicoCentauri
Copy link
Contributor

@HaoZeke do you have time to finish this? Would be aweseom to have faster tests!

@HaoZeke HaoZeke requested a review from Luthaf October 16, 2025 04:41
@HaoZeke
Copy link
Member Author

HaoZeke commented Oct 16, 2025

cscs-ci run

@RMeli
Copy link
Member

RMeli commented Oct 16, 2025

cscs-ci run

Copy link
Contributor

@PicoCentauri PicoCentauri left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines +46 to +47
# For tox to use uv..
ENV_INSTALLER: "uv pip"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is for pip in tox to use uv, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

yup

Co-authored-by: rmeli <rmeli@users.noreply.github.com>
@HaoZeke HaoZeke requested review from PicoCentauri and RMeli October 16, 2025 09:33
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.

4 participants