search

move-coop / parsons

A python library of connectors for the progressive community.
Other
255 stars 125 forks source link

Simpler use of pip cache in actions #986

Closed bmos closed 4 months ago

bmos commented 5 months ago

GitHub recommends using this approach: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows https://github.com/actions/setup-python#caching-packages-dependencies

That being said, what you have is clearly working so this is not an important change.

shaunagm commented 4 months ago

Thank you! This looks good to me, although I'll admit I'm just trusting what you and github say is the best route forward - I am whatever the opposite of a caching expert is.

Going to wait to merge until after I'm done doing PR review for today as sometimes changes to the tests mess with open PRs in inexplicable ways