-
Notifications
You must be signed in to change notification settings - Fork 6
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
Refactor Corda network DSL to produce a generic lib module? #38
Comments
Thanks @manosbatsis for this proposal. As you say the IMO, the Braid support can be dropped out as there is no clear roadmap on this project since a while ago. So that would simplify things for you since there would be no need for an extra module for the moment. We could always extend it later to add Braid support if needed but I believe you can safely proceed without it. How that sounds for you? |
Seems i (optionally) need braid too. I'll try to use the module dependency as is, then submit an MR with what i had to change for me to do so and we can take it from there - if you like it we can update/merge it, no strings attached. Thanks! |
Why would you need Braid? |
Cordite! |
Testcontainers support in corda-testacles currently requires the result of cordform/deployNodes as input. I've wanted to add testcontainers support for more "ad hoc" networks using a DSL for a while now. which obviously has a considerable overlap with web3j-corda's test module.
Instead of me essentially duplicating
org.web3j.corda.network.CordaNetwork
, would you be interested in a PR that breaks-up the existingweb3j-corda-test
to add an additional module, one providing a more generic/vanilla DSL library for others to extend? My understanding is such a module would leave out the focus on braid and hooks to RPC-enabled flows, those would remain inweb3j-corda-test
. This new module should allow me to reuse/extend the DSL implementation and i'd rather help maintaining a common reusable codebase than replicating "my own".The text was updated successfully, but these errors were encountered: