Skip to content
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

[recnet-release-action] cache gh action build output #330

Merged
merged 3 commits into from
Oct 16, 2024

Conversation

swh00tw
Copy link
Collaborator

@swh00tw swh00tw commented Oct 14, 2024

Description

Cache the output of nx build recnet-release-action using the hash of source code in libs/recnet-release-action/src to accelerate gh action

Related Issue

Notes

Successful run 1 (cache miss): https://github.com/lil-lab/recnet/actions/runs/11335345342/attempts/1
Successful run 2 (cache hit): https://github.com/lil-lab/recnet/actions/runs/11335345342

Test

Screenshots (if appropriate):

You can see in the first run, cache doesn't hit, and the action start downloading node packages and compiling the typescript files.

Screenshot 2024-10-14 at 5 41 54 PM

And it saves the output in cache in the end of first successful run.

Screenshot 2024-10-14 at 5 43 45 PM

In the second run, you can see it did hit the cache and skip the downloading and compiiing steps
Screenshot 2024-10-14 at 5 44 37 PM

TODO

  • Clear console.log or console.error for debug usage
  • Update the documentation recnet-docs if needed

Copy link

vercel bot commented Oct 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
recnet ⬜️ Ignored (Inspect) Oct 14, 2024 9:40pm
recnet-docs ⬜️ Ignored (Inspect) Oct 14, 2024 9:40pm

@swh00tw swh00tw changed the base branch from master to dev October 14, 2024 21:46
@swh00tw swh00tw merged commit bff532f into dev Oct 16, 2024
4 checks passed
swh00tw added a commit that referenced this pull request Oct 17, 2024
## RecNet auto-release action
This is an auto-generated PR by recnet-release-action 🤖
Please make sure to test your changes in staging before merging. 
## Related Issues
- #60
- #268
## Related PRs
- #331
- #330
- #328
- #326
## Staging links
recnet-web:
[https://vercel.live/link/recnet-git-dev-recnet-542617e7.vercel.app](https://vercel.live/link/recnet-git-dev-recnet-542617e7.vercel.app)
recnet-api:
[https://dev-api.recnet.io/api](https://dev-api.recnet.io/api)
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.

2 participants