-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
ci: Use turbo for build cache (no-changelog) #9696
Conversation
Last attempt: #7805 |
9480a36
to
5e94e0a
Compare
I don't see a significant difference between the reruns here: https://github.com/n8n-io/n8n/actions/runs/9468790143 |
|
58952b3
to
6b8de45
Compare
00a4fea
to
0bc41c1
Compare
I had some troubles setting up the |
0bc41c1
to
adc650e
Compare
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.
The workflow changes look good. I don't know anything about the config format, but it looks reasonable and I tried it locally and it works.
|
4 flaky tests on run #5808 ↗︎Details:
|
Test | Artifacts | |
---|---|---|
NDV > should not retrieve remote options when required params throw errors |
Screenshots
Video
|
|
NDV > Stop listening for trigger event from NDV |
Screenshots
Video
|
10-undo-redo.cy.ts • 2 flaky tests
Test | Artifacts | |
---|---|---|
Undo/Redo > should undo/redo adding nodes |
Test Replay
Screenshots
Video
|
|
Undo/Redo > should undo/redo adding connected nodes |
Test Replay
Screenshots
Video
|
Review all test suite changes for PR #9696 ↗︎
|
adc650e
to
2b05991
Compare
✅ All Cypress E2E specs passed |
Got released with |
This PR setups a remote cache for turbo-repo using a local server that backs up the cache using Github Actions Cache service.
Once this is working,
build
,test
, andlint
tasks should significantly speed up.Related tickets and issues
https://linear.app/n8n/issue/N8N-4262
Review / Merge checklist