-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(ci): enable cache in setup-python #682
Conversation
I haven't found this to make a massive difference in my cases, and the workflow is more complicated (more lines) so not really worth it. |
This is useful in the rare case of installing a library that takes an inordinate amount of time to install or using many libraries. However I'm also not sure if it should be in the template. (How about commenting it out or making this optional then?) I would appreciate it if you could take a look at other PRs as well, thank you |
with: | ||
python-version: ${{ matrix.python-version }} | ||
- uses: snok/install-poetry@v1.3.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just realised that this action uses the deprecated install script, and that the solution you suggest is the recommended one 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good otherwise
Thanks for the effort and your patience! |
https://github.com/actions/setup-python/blob/main/docs/advanced-usage.md#caching-packages
This may speed up the test slightly. (although it will take longer to install poetry anyway)