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

Updated rules_nodejs for wasm_bindgen rules #890

Merged
merged 2 commits into from
Aug 25, 2021

Conversation

UebelAndre
Copy link
Collaborator

@UebelAndre UebelAndre commented Aug 13, 2021

rules_nodejs made some updates to the names of some symbols. This PR picks those up so rules_rust may also benefit.

@google-cla google-cla bot added the cla: yes label Aug 13, 2021
@UebelAndre
Copy link
Collaborator Author

This PR is waiting on the actual release of rules_nodejs 4.0.0

@UebelAndre
Copy link
Collaborator Author

Also, it seems rules_nodejs no longer supports Bazel version 4.0.0 so until rules_rust upgrades it's min version, the latest release of rules_nodejs will not be usable by the rust_wasm_bindgen rule...

@UebelAndre
Copy link
Collaborator Author

Actually, rebasing on top of #865 works since in that PR, the rules_nodejs repositories are not used when testing the Bazel min version (which is only done on the core rules and tests, not examples).

@UebelAndre
Copy link
Collaborator Author

With rules_nodejs 4.0.0 out now, this PR can be merged.

@UebelAndre UebelAndre merged commit d2bf64f into bazelbuild:main Aug 25, 2021
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.

None yet

2 participants