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 sourcemap PR compile error #7162

Merged
merged 1 commit into from
Jan 30, 2024
Merged

Conversation

timneutkens
Copy link
Member

Description

Fixes the error on vercel/next.js#61369

Test run: https://github.com/vercel/turbo/blob/08e09624524425db1530381941d1fedbed27fe4f/crates/turbopack-dev/src/ecmascript/content_entry.rs#L44

Excerpt:

  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntries` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:856:35
      |
  856 |                     let entries = content.entries().await?;
      |                                   ^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:857:33
      |
  857 |                     let entry = entries.get(&ModuleId::String(module).cell().await?);
      |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:858:37
      |
  858 |                     let map = match entry {
      |                                     ^^^^^ private type
  
  error: could not compile `next-swc-napi` (lib) due to 3 previous errors

Testing Instructions

Copy link

vercel bot commented Jan 30, 2024

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

Name Status Preview Comments Updated (UTC)
examples-basic-web 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-gatsby-web 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-kitchensink-blog 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-nonmonorepo 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-svelte-web 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-tailwind-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jan 30, 2024 9:55am
examples-vite-web 🔄 Building (Inspect) Visit Preview Jan 30, 2024 9:55am
rust-docs ❌ Failed (Inspect) Jan 30, 2024 9:55am
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 30, 2024 9:55am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Jan 30, 2024 9:55am
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Jan 30, 2024 9:55am

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @timneutkens and the rest of your teammates on Graphite Graphite

Copy link
Contributor

🟢 Turbopack Benchmark CI successful 🟢

Thanks

Copy link
Contributor

✅ This change can build next-swc

Copy link
Contributor

github-actions bot commented Jan 30, 2024

⚠️ CI failed ⚠️

The following steps have failed in CI:

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

See workflow summary for details

@sokra sokra merged commit eaea57c into main Jan 30, 2024
52 of 55 checks passed
@sokra sokra deleted the 01-30-Fix_sourcemap_PR_compile_error branch January 30, 2024 11:08
sokra pushed a commit to vercel/next.js that referenced this pull request Jan 30, 2024
# Turbopack changes
* vercel/turborepo#7160 <!-- Will Binns-Smith - Make
`EcmascriptDevChunkContent` public to other crates -->
* vercel/turborepo#7162 <!-- Tim Neutkens - Fix
sourcemap PR compile error -->
* vercel/turborepo#7163 <!-- Donny/강동윤 - fix: Revert
`preset-env` mode of `styled-jsx` in turbopack -->


### What?

Update turbopack

### Why?

To apply vercel/turborepo#7163

### How?



Closes PACK-2323
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 25, 2024
### Description

Fixes the error on #61369

Test run:
https://github.com/vercel/turbo/blob/7799d37728ae34be5f3aedd227c955067e16c274/crates/turbopack-dev/src/ecmascript/content_entry.rs#L44

Excerpt:

```
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntries` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:856:35
      |
  856 |                     let entries = content.entries().await?;
      |                                   ^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:857:33
      |
  857 |                     let entry = entries.get(&ModuleId::String(module).cell().await?);
      |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:858:37
      |
  858 |                     let map = match entry {
      |                                     ^^^^^ private type
  
  error: could not compile `next-swc-napi` (lib) due to 3 previous errors

```

<!--
  ✍️ Write a short summary of your work.
  If necessary, include relevant screenshots.
-->

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
### Description

Fixes the error on #61369

Test run:
https://github.com/vercel/turbo/blob/d14febb4f93d20bb21fb821a2326ed7339163b0d/crates/turbopack-dev/src/ecmascript/content_entry.rs#L44

Excerpt:

```
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntries` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:856:35
      |
  856 |                     let entries = content.entries().await?;
      |                                   ^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:857:33
      |
  857 |                     let entry = entries.get(&ModuleId::String(module).cell().await?);
      |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:858:37
      |
  858 |                     let map = match entry {
      |                                     ^^^^^ private type
  
  error: could not compile `next-swc-napi` (lib) due to 3 previous errors

```

<!--
  ✍️ Write a short summary of your work.
  If necessary, include relevant screenshots.
-->

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
### Description

Fixes the error on #61369

Test run:
https://github.com/vercel/turbo/blob/35de4a076ce6642fa502b736f101a91e3386e5f9/crates/turbopack-dev/src/ecmascript/content_entry.rs#L44

Excerpt:

```
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntries` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:856:35
      |
  856 |                     let entries = content.entries().await?;
      |                                   ^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:857:33
      |
  857 |                     let entry = entries.get(&ModuleId::String(module).cell().await?);
      |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:858:37
      |
  858 |                     let map = match entry {
      |                                     ^^^^^ private type
  
  error: could not compile `next-swc-napi` (lib) due to 3 previous errors

```

<!--
  ✍️ Write a short summary of your work.
  If necessary, include relevant screenshots.
-->

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 1, 2024
### Description

Fixes the error on #61369

Test run:
https://github.com/vercel/turbo/blob/8891f6b758d459dd030d40d76dc9c8c16f3966db/crates/turbopack-dev/src/ecmascript/content_entry.rs#L44

Excerpt:

```
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntries` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:856:35
      |
  856 |                     let entries = content.entries().await?;
      |                                   ^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:857:33
      |
  857 |                     let entry = entries.get(&ModuleId::String(module).cell().await?);
      |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ private type
  
  error: type `turbopack_binding::turbopack::turbopack_dev::ecmascript::content_entry::EcmascriptDevChunkContentEntry` is private
     --> packages/next-swc/crates/napi/src/next_api/project.rs:858:37
      |
  858 |                     let map = match entry {
      |                                     ^^^^^ private type
  
  error: could not compile `next-swc-napi` (lib) due to 3 previous errors

```

<!--
  ✍️ Write a short summary of your work.
  If necessary, include relevant screenshots.
-->

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants