-
Notifications
You must be signed in to change notification settings - Fork 436
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
Support for Bazel #197
Comments
Hey @igorgatis, Indeed. Currently working on improbable-eng/ts-protoc-gen#77. Once we have that, grpc-web is the natural next step. Having said that, your help would be greatly appreciated. If you've got experience with Bazel or want to dig in, please do. We can use this channel to figure out the details. |
Hey @easyCZ. Just came across this issue looking for bazel support for grpc-web. Just FYI I noticed that this pull request improbable-eng/ts-protoc-gen#84 has been merged and so the ts-protoc-gen bazel rule is I think complete but issue improbable-eng/ts-protoc-gen#77 hasn't been closed yet. Thought I'd just flag it here as you're waiting for improbable-eng/ts-protoc-gen#77 to move forward with bazel support for grpc-web - which I selfishly want :) |
Isn't there already bazel support? It would be nice if rules-typescript allowed ts code to import the files generated by ts-protoc-gen in a cleaner way rather than having to reference gen files or use path aliases. But it's still supported and works. Maybe I'm missing something? ¯\_(ツ)_/¯ |
Do we need both |
It would be great to have rules for Bazel, specially if they work with https://github.com/bazelbuild/rules_typescript
The text was updated successfully, but these errors were encountered: