Skip to content

chore(ci): make the cache not depends on generated files #458

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

Merged
merged 5 commits into from
Apr 29, 2022

Conversation

millotp
Copy link
Collaborator

@millotp millotp commented Apr 29, 2022

🧭 What and Why

This is an attempt to fix the CI on #451, the cache depends on generated files but the cache key are generated before the generation happens, thus in the Post Cache job the cache-key is incorrect.

Changes included:

  • Remove all generated code from cache-key and replace with higher dependencies

🧪 Test

CI

@millotp millotp self-assigned this Apr 29, 2022
@netlify
Copy link

netlify bot commented Apr 29, 2022

Deploy Preview for api-clients-automation ready!

Name Link
🔨 Latest commit 8351ba4
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/626bfdde089cf7000831ffe3
😎 Deploy Preview https://deploy-preview-458--api-clients-automation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@algolia-bot
Copy link
Collaborator

algolia-bot commented Apr 29, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.

@shortcuts
Copy link
Member

Maybe rebase this PR on top of #451 so we know if it works directly?

@millotp
Copy link
Collaborator Author

millotp commented Apr 29, 2022

good idea !

@millotp millotp force-pushed the chore/cache-no-gen-file branch from 84075cf to 8b9b4f1 Compare April 29, 2022 13:49
@millotp millotp changed the base branch from main to fix/search-spec-cleanup April 29, 2022 13:49
@millotp millotp force-pushed the chore/cache-no-gen-file branch from 8b9b4f1 to 52f6850 Compare April 29, 2022 13:52
@shortcuts
Copy link
Member

I can already smell victory here

@shortcuts
Copy link
Member

purrfect

@millotp millotp force-pushed the chore/cache-no-gen-file branch from 3975e3b to 8351ba4 Compare April 29, 2022 15:01
@millotp millotp merged commit 5cc00af into fix/search-spec-cleanup Apr 29, 2022
@millotp millotp deleted the chore/cache-no-gen-file branch April 29, 2022 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants