Skip to content

Conversation

@ghubertpalo
Copy link
Collaborator

Add a CLI runner with a float String -> u64 parser.

@github-actions
Copy link

github-actions bot commented Jun 21, 2022

Unit Test Results

213 tests  +5   213 ✔️ +5   6m 55s ⏱️ -3s
  17 suites ±0       0 💤 ±0 
    6 files   ±0       0 ±0 

Results for commit c451a36. ± Comparison against base commit a768c45.

♻️ This comment has been updated with latest results.

@jpraynaud jpraynaud force-pushed the jpraynaud/250-chain-observer branch from 9981f96 to f44dfc6 Compare June 21, 2022 13:28
@ghubertpalo ghubertpalo force-pushed the greg/250/chain-observer branch from 04eba8b to dae68cb Compare June 21, 2022 14:58
@ghubertpalo ghubertpalo requested review from Alenar and jpraynaud June 22, 2022 07:25
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Comment on lines 59 to 60
.arg("--cardano-mode")
.arg("--testnet-magic 42")
Copy link
Member

@jpraynaud jpraynaud Jun 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we need to configure these options:

  • --cardano-mode only necessary on the devnet
  • --testnet-magic ${TESTNET_MAGIC_ID} only necessary on the devnet and the testnet (with a different ${TESTNET_MAGIC_ID} for each network)
  • --mainnet used instead on the mainnet

These configuration options will be also needed in the launch_stake_distribution cli call

@jpraynaud jpraynaud force-pushed the jpraynaud/250-chain-observer branch from f44dfc6 to bdbdabf Compare June 22, 2022 09:25
Base automatically changed from jpraynaud/250-chain-observer to main June 22, 2022 09:45
@ghubertpalo ghubertpalo force-pushed the greg/250/chain-observer branch from 1ff4525 to c451a36 Compare June 22, 2022 12:23
@ghubertpalo ghubertpalo merged commit 9d611e7 into main Jun 22, 2022
@ghubertpalo ghubertpalo deleted the greg/250/chain-observer branch June 22, 2022 14:11
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.

3 participants