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

V2 #193

Merged
merged 284 commits into from
Nov 16, 2021
Merged

V2 #193

merged 284 commits into from
Nov 16, 2021

Conversation

pcj
Copy link
Member

@pcj pcj commented Nov 16, 2021

  • Large rewrite with gazelle support

Paul Johnston and others added 29 commits September 20, 2021 18:17
* Add c++ client/server example
* Elide feature_test
* Migrate builtin_test
* Migrate scalapb_test
* Migrate grpc_test
* Add initial proto_repository.bzl
* Vendor bazel-gazelle PR 1033
* YConfig parsing
* Update readme
Previously the mappings were not accouting for the case of being run
in an external workspace.  To fix that, it is necessary to thread in
the config.WorkDir property.  If we are running in a directory like
/.../external/foo, the output mappings become prefixed with
"external/foo/..."
This was an experiment within python prior to integration with the rules_python gazelle generator.
)

* Rename grpc:grpc-go:protoc-gen-go -> golang:protobuf:protoc-gen-go
* Introduce LanguageRuleConfig.Resolve directive
* Add list_repository_tools_srcs
* Gazelle apply
* Buildfier fixes
This change add a new repository rule that works like go_repository.  There is quite a bit of changes around rule resolution and dependency management.
* Initial ts_proto_library
* Rewrite with tsc action
* Add transitive support
* Add proto_ts_library.go
* Use genrule script instead of go_binary
* Initial proto_ts_library impl
* Update example with nestJs/gRPC
* Pass args to the tsc compile action
* Fix output file naming for protoc-gen-ts-proto
* Upgrade bazel-gazelle, add mergemappings test
* Migrate mappings (string_dict) -> output_mappings (string_list)
* Fix deps resolution for proto_java_library
@pcj pcj merged commit 7c95feb into master Nov 16, 2021
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.

1 participant