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

ci: Set up Turborepo cache again (no-changelog) #7805

Closed
wants to merge 3 commits into from

Conversation

ivov
Copy link
Contributor

@ivov ivov commented Nov 24, 2023

This PR sets up this GH action so we can use the GH runner's cache service for Turborepo runs.

The last time we had to revert but the full logs of the issue are no longer available so we might have to end up triggering the same issue again for debugging.

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Nov 24, 2023
Comment on lines +11 to +14
"build:backend": {},
"build:frontend": {},
"test:backend": {},
"test:frontend": {},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this alone is enough. we need to define what workspaces to run the build command in. We could try updating https://github.com/n8n-io/n8n/blob/master/package.json#L28 to use turbo run --filter= instead of pnpm --filter=, but unlike pnpm, turbo will still run the tests for editor, because n8n depends on that package.

@ivov
Copy link
Contributor Author

ivov commented Feb 1, 2024

This PR was lost in the shuffle. Closing for now, let's revisit when we have time for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants