search

MDAnalysis / UserGuide

User Guide for MDAnalysis
https://userguide.mdanalysis.org
22 stars 33 forks source link

include date in cache key #359

Closed mikemhenry closed 9 months ago

mikemhenry commented 9 months ago

This will cache the env + the package downloads. The date is included in the cache key so it will never be older than a day. This will still save some time/carbon when running CI, but will ensure that scheduled runs will get a "fresh" env.

📚 Documentation preview 📚: https://mdanalysisuserguide--359.org.readthedocs.build/en/359/

mikemhenry commented 9 months ago

Ah when I looked at #355 it was just that file -- but your changes look correct!

IAlibay commented 9 months ago

Ah when I looked at #355 it was just that file -- but your changes look correct!

Yeah looks like I goofed in #355 🙈 sorry!

IAlibay commented 9 months ago

CI failure is expected, see #357

IAlibay commented 9 months ago

Thanks for the quick solution @mikemhenry !