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

Export closure_proto_aspect and document closure_grpc_web_library #278

Merged
merged 2 commits into from
Oct 16, 2018

Conversation

Yannic
Copy link
Contributor

@Yannic Yannic commented Aug 4, 2018

Exposing the aspect is required for the implementation of
closure_grpc_web_library in https://github.com/grpc/grpc-web/

@Yannic
Copy link
Contributor Author

Yannic commented Aug 4, 2018

Exposing the aspect is required for the implementation of
closure_grpc_web_library in https://github.com/grpc/grpc-web/
@Yannic Yannic force-pushed the expose_closure_proto_aspect branch from 53f2cab to 16e0cde Compare August 6, 2018 17:50
@Yannic Yannic changed the title Export closure_proto_aspect Export closure_proto_aspect and document closure_grpc_web_library Aug 6, 2018
@pcj
Copy link
Member

pcj commented Sep 20, 2018

LGTM. Please rebase to master. @jart or @dslomov can you review?

@v48
Copy link

v48 commented Oct 13, 2018

This sounds awesome! I would like to try closure_grpc_web_library for my project Is there a possibility to merge this soon? Im currently generating files with a separate script it would be awesome to use bazel instead.

@v48
Copy link

v48 commented Oct 15, 2018

@Yannic Would it be possible to get this change in soon? I would really like to try this bazel rule in my project.

@v48
Copy link

v48 commented Oct 16, 2018

@dslomov Thanks for rebasing and updating 🎉 , is there an eta on when this will be released?

@dslomov dslomov merged commit acad969 into bazelbuild:master Oct 16, 2018
ptmphuong pushed a commit to ptmphuong/rules_closure that referenced this pull request Dec 9, 2022
…zelbuild#278)

Exposing the aspect is required for the implementation of
closure_grpc_web_library in https://github.com/grpc/grpc-web/
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.

5 participants