carpentries / styles

Styles for The Carpentries lessons. No README to avoid merge conflicts with lessons. Demo 👇
https://carpentries.github.io/lesson-example
Other
84 stars 94 forks source link

Update cache mechanism for R packages #605

Open zkamvar opened 3 years ago

zkamvar commented 3 years ago

Our cache system is in the workflows is not working and that's because I did not have a good model of how the caching worked previously. Take the following code:

https://github.com/carpentries/styles/blob/da771119b6c4adc61dea3e33786a3c8179600cb1/.github/workflows/website.yml#L55-L79

Line 60 depends on the files created by Lines 77 and 78, respectively. What needs to happen is that, of these three steps, "Restore R Cache" should go at the end. I wrote a bit about this the last time I had to wrestle this issue: https://github.com/zkamvar/actions/pull/13#issuecomment-850696546