-
Notifications
You must be signed in to change notification settings - Fork 277
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
feat(core-api): add weaver protobuf codegen #1556 #1759
Conversation
@VRamakrishna What do you think of this? If it looks good I'll set up the auto-publishing github workflow for a crate. |
287abee
to
a5520fd
Compare
packages/cactus-core-api/src/main/rust/generated/proto-rs/common.access_control.rs
Outdated
Show resolved
Hide resolved
How do we now build the relay as a standalone component using the code generated from the protos? Also, can you add a |
a5520fd
to
49dc56f
Compare
@VRamakrishna Adding the readme in the scope of another issue that we had open for a while, stay tuned for the follow-up PR with that: #833 Added a Makefile so now you can run |
Rust code now gets generated from the Weaver protocol buffer definitions Next step is to have the automated publishing set up. To test the rust build, you need to run either one of the following: `yarn lerna run proto:protoc-gen-rust` (from the project root) or a regular cargo build from the core-api package directory, e.g.: ```sh cd packages/cactus-core-api/ cargo build ``` Fixes hyperledger#1556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
49dc56f
to
794207a
Compare
feat(core-api): add weaver protobuf codegen #1556
Rust code now gets generated from the Weaver protocol buffer definitions
Next step is to have the automated publishing set up.
To test the rust build, you need to run either one of the following:
yarn lerna run proto:protoc-gen-rust
(from the project root)or a regular cargo build from the core-api package directory, e.g.:
cd packages/cactus-core-api/ cargo build
Fixes #1556
Signed-off-by: Peter Somogyvari peter.somogyvari@accenture.com