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

Allow "commonjs" as import_style #700

Closed
wants to merge 3 commits into from
Closed

Allow "commonjs" as import_style #700

wants to merge 3 commits into from

Conversation

negator
Copy link

@negator negator commented Jan 7, 2020

Allows the specification of "commonjs" as an import style, as documented here: https://github.com/grpc/grpc-web#import-style

Allows the specification of "commonjs" as an import style, as documented here: https://github.com/grpc/grpc-web#import-style
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jan 7, 2020

CLA Check
The committers are authorized under a signed CLA.

@stanley-cheung
Copy link
Collaborator

See the discussion on #260

cc @Yannic

@negator
Copy link
Author

negator commented Jan 17, 2020

Thanks for the pointer. Looks like its more complicated than simply allowing closure_grpc_web_library to accept commonjs, due to incomplete/non-existent support in rules_closure rules. Are there any suggested workarounds, if we plan to use closure_grpc_web_library exclusively?

Yannic added a commit to Yannic/grpc-web that referenced this pull request Mar 3, 2020
stanley-cheung pushed a commit that referenced this pull request Mar 19, 2020
@stanley-cheung
Copy link
Collaborator

Closing this for now. I have not kept up with the changes on the bazel side. If this becomes relevant again, please re-open.

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