-
Notifications
You must be signed in to change notification settings - Fork 0
Nightly CI Tests #13
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
Nightly CI Tests #13
Conversation
~/.npm | ||
${{ github.workspace }}/.next/cache | ||
# Generate a new cache whenever packages or source files change. | ||
key: ${{ runner.os }}-nextjs-${{ hashFiles('yarn.lock') }}-${{ hashFiles('**/*.js', '**/*.jsx', '**/*.ts', '**/*.tsx') }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good idea 🤔
I wonder if I can use this in the SDK repo to re-use archived builds from different commits if there are no source changes. Would cut down the majority of the workflow time.
with: | ||
path: | | ||
~/.npm | ||
${{ github.workspace }}/.next/cache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's in the .next/cache
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a build cache or Next. Nexst complains if you don't use it in CI enviornments, I guess, so I cached it!
key: ${{ runner.os }}-nextjs-${{ hashFiles('yarn.lock') }}-${{ hashFiles('**/*.js', '**/*.jsx', '**/*.ts', '**/*.tsx') }} | ||
# If source files changed but packages didn't, rebuild from a prior cache. | ||
restore-keys: | | ||
${{ runner.os }}-nextjs-${{ hashFiles('yarn.lock') }}- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you accidentally left a hanging -
at the end here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it still matches the top key up until the second hasFiles. I actually copied this from "the manual", so it's legit!
Add nightly CI execution of Firebse JS SDK tests exercised by Playwright.
The CI workflow caches:
The CI workflow will run:
workflow_dispatch
)Future configuration within the GitHub repo will email the Firebase JS SDK team if there are failures.