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

Fixing the Wasm blocker for Parser #494

Closed
wants to merge 2 commits into from
Closed

Fixing the Wasm blocker for Parser #494

wants to merge 2 commits into from

Conversation

anurbol
Copy link
Contributor

@anurbol anurbol commented Dec 10, 2019

Rust has a bug when compiling a project for Wasm (with --target wasm32-unknown-unknown flag), these changes are a workaround.

These changes are only supposed to fix the SWC Parser, not the whole SWC project. This is because I only need the Parser at the moment, also fixing the whole SWC Project would be a nightmare for me, a newbie in SWC. Maybe later.

@kdy1
Copy link
Member

kdy1 commented Dec 10, 2019

bors r+

bors bot added a commit that referenced this pull request Dec 10, 2019
494: Fixing the Wasm blocker for Parser r=kdy1 a=anurbol

Rust has a [bug](rust-lang/rust#64450) when compiling a project for Wasm (with `--target wasm32-unknown-unknown` flag), these changes are a workaround.

These changes are only supposed to fix the SWC Parser, not the whole SWC project. This is because I only need the Parser at the moment, also fixing the whole SWC Project would be a nightmare for me, a newbie in SWC. Maybe later.

Co-authored-by: Nurbol Alpysbayev <anurbol@gmail.com>
@anurbol
Copy link
Contributor Author

anurbol commented Dec 10, 2019

I can't understand what's wrong there for the linter...

@kdy1
Copy link
Member

kdy1 commented Dec 10, 2019

Can you fix tests too?

@bors
Copy link
Contributor

bors bot commented Dec 10, 2019

Build failed

@anurbol
Copy link
Contributor Author

anurbol commented Dec 10, 2019

@kdy Yes, one second, God how did I overlooked that...

@anurbol anurbol closed this Dec 10, 2019
@swc-project swc-project locked as resolved and limited conversation to collaborators Nov 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants