Skip to content

Conversation

@re-taro
Copy link
Contributor

@re-taro re-taro commented Nov 21, 2025

Changes

  • Fix members in workspace

close #329

@re-taro
Copy link
Contributor Author

re-taro commented Nov 21, 2025

@marco-ippolito

Sorry I fixed it!

@marco-ippolito
Copy link
Member

Im afraid this change will be ovewritten during deps update
@kdy1 can you take a look

@kdy1
Copy link
Member

kdy1 commented Nov 22, 2025

Yeap, I’ll take a look on Monday (I’ll be flying over the weekend)

@marco-ippolito
Copy link
Member

@re-taro maybe we could write it as a something.patch and apply the patch in the CI, so it does not get overriden. I think the issue is that we have a part of swc (binding)

Copy link
Member

@kdy1 kdy1 left a comment

Choose a reason for hiding this comment

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

➜  swc git:(fix/swc) ✗ cargo check
error: failed to load manifest for workspace member `/Users/kdy1/projects/amaro/deps/swc/bindings/binding_core_node`
referenced via `bindings/*` by workspace at `/Users/kdy1/projects/amaro/deps/swc/Cargo.toml`

Caused by:
  failed to parse manifest at `/Users/kdy1/projects/amaro/deps/swc/bindings/binding_core_node/Cargo.toml`

Caused by:
  error inheriting `backtrace` from workspace root manifest's `workspace.dependencies.backtrace`

Caused by:
  `dependency.backtrace` was not found in `workspace.dependencies`

I think it's using an old version of SWC clone?

@marco-ippolito
Copy link
Member

➜  swc git:(fix/swc) ✗ cargo check
error: failed to load manifest for workspace member `/Users/kdy1/projects/amaro/deps/swc/bindings/binding_core_node`
referenced via `bindings/*` by workspace at `/Users/kdy1/projects/amaro/deps/swc/Cargo.toml`

Caused by:
  failed to parse manifest at `/Users/kdy1/projects/amaro/deps/swc/bindings/binding_core_node/Cargo.toml`

Caused by:
  error inheriting `backtrace` from workspace root manifest's `workspace.dependencies.backtrace`

Caused by:
  `dependency.backtrace` was not found in `workspace.dependencies`

I think it's using an old version of SWC clone?

See #333

Copy link
Member

@kdy1 kdy1 left a comment

Choose a reason for hiding this comment

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

This PR has a problem even if I rebase to the relevant branch.

➜  swc git:(fix/swc) ✗ cargo check
error: failed to load manifest for workspace member `/Users/kdy1/projects/amaro/deps/swc/bindings/binding_core_node`
referenced via `bindings/*` by workspace at `/Users/kdy1/projects/amaro/deps/swc/Cargo.toml`

Caused by:
  failed to load manifest for dependency `swc_core`

Caused by:
  failed to read `/Users/kdy1/projects/amaro/deps/swc/crates/swc_core/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

This is not a correct direction either. Why are you trying to do this?

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.

swc update workflow is broken

3 participants