chore: API Build cache in CI pipeline #14929
Conversation
|
@utkershrajvenshi is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
github-actions
bot
added
api
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
Graphite Automations"Add community label" took an action on this PR • (05/07/24)1 label was added to this PR based on Keith Williams's automation. "Add consumer team as reviewer" took an action on this PR • (05/07/24)1 reviewer was added to this PR based on Keith Williams's automation. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
| key-4: ${{ github.sha }} | ||
| with: | ||
| path: | | ||
| ${{ github.workspace }}/apps/api/v2/.next |
There was a problem hiding this comment.
v2 is not a Next.js app but rather a NestJS app. What was the intention of this line?
There was a problem hiding this comment.
That seems to be an oversight from my end. Sorry about that.
One thing I would like to run by you is that while running build command for api/v2 the build was stored in ./dist folder. So should this and **/.turbo/** be even required to pass under path?
There was a problem hiding this comment.
I don't believe so. Think we should be good with the dist line
There was a problem hiding this comment.
Super! I'll make the relevant changes 🙏
|
Hi @keithwillcode , are there any other changes to be done for this PR? |
What does this PR do?
Added cache retrieval steps for api v1 and api v2 in production builds github workflow run
Fixes #14921
Type of change
Mandatory Tasks
Checklist