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
chore: differentiate rust changes from building turbo #8001
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🟢 Turbopack Benchmark CI successful 🟢Thanks |
🟢 CI successful 🟢Thanks |
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.
FWIW, I think it would make sense to exclude the turborepo-repository
tests as a check before release, since it is independently released. We do this in CI also:
turbo/.github/workflows/test.yml
Line 395 in e63b1a8
TURBO_API= turbo run check-types test --filter=docs --filter="!turborepo-repository" --filter={./packages/*}...[${{ github.event.pull_request.base.sha || 'HEAD^1' }}] --color --env-mode=strict |
and turborepo-repository
is independently tested in its own workflow
Description
This PR adds a new dummy task that will be invalidated whenever turborepo related Rust code changes.
I noticed we were building the
turbo
binary when running our JS tests. This is due toturborepo-repository
depending oncli#build
, but I believe that we really just want it invalidate on any Rust code changes.Testing Instructions
Verify that updating a Rust file causes a cache miss for both
turbo build --filter=cli
andturbo build --filter=turborepo-repository
.Closes TURBO-2837