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.

Sign up for GitHub

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

Jump to bottom

ci: remove daily turbopack next.js integration test workflow #6588

Merged
merged 3 commits into from
Dec 4, 2023
Merged

ci: remove daily turbopack next.js integration test workflow #6588

merged 3 commits into from
Dec 4, 2023

Conversation

ForsakenHarmony
Copy link
Contributor

@ForsakenHarmony ForsakenHarmony commented Nov 27, 2023 •
edited by padmaia
Loading

Description

This is done in the next.js repo now: vercel/next.js#58965

Closes PACK-2040

@ForsakenHarmony ForsakenHarmony requested review from a team as code owners November 27, 2023 13:15
@vercel Vercel
Copy link

vercel bot commented Nov 27, 2023 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-gatsby-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-kitchensink-blog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-svelte-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
examples-vite-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
rust-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 4, 2023 3:36pm

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 27, 2023 •
edited
Loading

🟢 Turbopack Benchmark CI successful 🟢

Thanks

@github-actions GitHub Actions
Copy link
Contributor

✅ This change can build next-swc

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 27, 2023 •
edited
Loading

⚠️ CI failed ⚠️

The following steps have failed in CI:

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

See workflow summary for details

tknickman
tknickman approved these changes Nov 27, 2023
View reviewed changes
Copy link
Member

@tknickman tknickman left a comment

Choose a reason for hiding this comment

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

👍🏼 from turborepo

@ForsakenHarmony ForsakenHarmony merged commit baf5c94 into main Dec 4, 2023
46 of 51 checks passed
@ForsakenHarmony ForsakenHarmony deleted the hrmny/remove-daily-turbopack-test branch December 4, 2023 16:12
@sokra sokra mentioned this pull request Dec 5, 2023
Merged
sokra added a commit to vercel/next.js that referenced this pull request Dec 5, 2023
@sokra
update turbopack (#59285)
2b47397 
* vercel/turborepo#6588 <!-- Leah - ci: remove daily
turbopack next.js integration test workflow -->
* vercel/turborepo#6666 <!-- Will Binns-Smith -
Turbopack css: treat `composes` and `@import` as urls when resolving -->
* vercel/turborepo#6701 <!-- Tobias Koppers - add
user level tracing -->
* vercel/turborepo#6616 <!-- Leah - feat(turbopack):
support "loading" WebAssembly injected as global varables in edge
runtime -->

Closes PACK-2086
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tknickman tknickman tknickman approved these changes

@mehulkar mehulkar mehulkar approved these changes

Assignees
No one assigned
Labels
area: ci
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ForsakenHarmony @mehulkar @tknickman