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

CAD-2717: cardano node service extensions #2443

Merged
merged 2 commits into from
Mar 4, 2021

Conversation

deepfire
Copy link
Contributor

@deepfire deepfire commented Mar 4, 2021

  1. Allow the topology path config option to also be a string.
  2. Add the executable option, to allow customising the way the node is started -- so that we can, for example, use cabal run -- cardano-node.

@deepfire deepfire force-pushed the cad-2712-cardano-node-service branch 2 times, most recently from 7dfd84b to e1e6de9 Compare March 4, 2021 01:00
@Jimbo4350 Jimbo4350 self-requested a review March 4, 2021 09:46
@deepfire deepfire force-pushed the cad-2712-cardano-node-service branch from e1e6de9 to c55a2f3 Compare March 4, 2021 11:13
@deepfire
Copy link
Contributor Author

deepfire commented Mar 4, 2021

bors r+

iohk-bors bot added a commit that referenced this pull request Mar 4, 2021
2443: CAD-2717: cardano node service extensions r=deepfire a=deepfire

1. Allow the `topology` path config option to also be a string.
2. Add the `executable` option, to allow customising the way the node is started -- so that we can, for example, use `cabal run -- cardano-node`.

Co-authored-by: Kosyrev Serge <serge.kosyrev@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 4, 2021

Build failed:

@deepfire
Copy link
Contributor Author

deepfire commented Mar 4, 2021

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 4, 2021

@iohk-bors iohk-bors bot merged commit f1ffe1f into master Mar 4, 2021
@iohk-bors iohk-bors bot deleted the cad-2712-cardano-node-service branch March 4, 2021 13:02
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.

None yet

3 participants