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

fix(turborepo): Rust integration test fixes #6142

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

NicholasLYang
Copy link
Contributor

@NicholasLYang NicholasLYang commented Oct 10, 2023

Description

Testing Instructions

Closes TURBO-1442

@vercel
Copy link

vercel bot commented Oct 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

11 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-cra-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-nonmonorepo ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm
turbo-site ⬜️ Ignored (Inspect) Visit Preview Oct 20, 2023 3:57pm

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

✅ This change can build next-swc

@github-actions
Copy link
Contributor

Linux Benchmark for e644d6b

Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 6358.61µs ± 28.14µs 6532.25µs ± 45.51µs +2.73% +0.41%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 6358.61µs ± 28.14µs 6532.25µs ± 45.51µs +2.73% +0.41%
bench_hmr_to_eval/Turbopack CSR/1000 modules 6095.95µs ± 36.54µs 6374.77µs ± 278.85µs +4.57%
bench_startup/Turbopack CSR/1000 modules 1240.11ms ± 2.81ms 1242.95ms ± 3.94ms +0.23%

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

⚠️ CI failed ⚠️

The following steps have failed in CI:

  • Turbopack Rust tests (mac/win, non-blocking)

See workflow summary for details

@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2023

🟢 Turbopack Benchmark CI successful 🟢

Thanks

@NicholasLYang NicholasLYang marked this pull request as ready for review October 19, 2023 16:47
@NicholasLYang NicholasLYang requested review from a team as code owners October 19, 2023 16:47
@NicholasLYang NicholasLYang force-pushed the fix/rust-integration-test-fixes branch from 1a1e611 to 6cdc9a6 Compare October 19, 2023 17:22
@NicholasLYang NicholasLYang enabled auto-merge (squash) October 19, 2023 17:26
@@ -332,6 +332,35 @@ jobs:
- name: Integration Tests
run: turbo run test --filter=turborepo-tests-integration --color --env-mode=strict --token=${{ secrets.TURBO_TOKEN }} --team=${{ vars.TURBO_TEAM }}

turborepo_integration_rust:
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be matrixed into the job above? It's 90% the same. An if: {{matrix.codepath == "rust"}} shuold do the trick to change the command that runs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't want this to be blocking, so I think a second job is best

Copy link
Contributor

Choose a reason for hiding this comment

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

Why not make it blocking? Seems like we're close to the end and should make it blocking at some point

@NicholasLYang NicholasLYang force-pushed the fix/rust-integration-test-fixes branch 2 times, most recently from e4a2900 to 9696b12 Compare October 19, 2023 18:11
@NicholasLYang NicholasLYang force-pushed the fix/rust-integration-test-fixes branch from 9696b12 to 1e3ff59 Compare October 20, 2023 15:56
@NicholasLYang NicholasLYang merged commit a930c83 into main Oct 20, 2023
@NicholasLYang NicholasLYang deleted the fix/rust-integration-test-fixes branch October 20, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants