-
Notifications
You must be signed in to change notification settings - Fork 76
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
Fetch cache in a single job #254
Fetch cache in a single job #254
Conversation
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
1 similar comment
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
382f58a
to
a9dd565
Compare
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
d15d3e3
to
20f1fc2
Compare
Removed references to cache which used time as the key and didn't speed up the CI
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
20f1fc2
to
0d3181a
Compare
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
1 similar comment
Links to coverage and verification reports for this PR (#254) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
This change moves fetching/setting up the cache to outside of the code-generating matrix.
The requests are made twice: once to fetch all the tests, and once to cache them; this should reduce the number of cache requests.