Cardano.Wallet.Shelley.Launch.BlockfrostSpec

[Unisay]

• I have changed the code such that --node-socket and --light --blockfrost-token-file options are mutually exclusive;
• added 2 more unit tests;

Usage: cardano-wallet serve [--listen-address HOST] 
                            (--node-socket FILE | --light
                              --blockfrost-token-file FILE) 
                            [--random-port | --port INT] 
                            [--tls-ca-cert FILE --tls-sv-cert FILE
                              --tls-sv-key FILE] 
                            (--mainnet | --testnet FILE | --staging FILE) 
                            [--database DIR] [--sync-tolerance DURATION] 
                            [--shutdown-handler] 
                            [--pool-metadata-fetching ( none | direct | SMASH-URL )]
                            [--token-metadata-server URL] 
                            [--trace-NAME SEVERITY]
  Serve API that listens for commands/actions.

Issue Number

ADP-1426

[HeinrichApfelmus]

Looks good to me! 😊 I find no obvious defects.

[Anviking]

I think this broke CI (hydra) on master 😶 => #3180

@Unisay I would recommend merging using bors r+/bors merge and not manually. Manual merges used to be disabled, but I think it was enabled recently not to block us in cases where we were struggling to get something merged with bors, but where it was clear that the failures weren't related to our PR.

I see there's no clear mention in the docs however, we should probably add it

[Unisay]

I think this broke CI (hydra) on master no_mouth => #3180

@Unisay I would recommend merging using bors r+/bors merge and not manually. Manual merges used to be disabled, but I think it was enabled recently not to block us in cases where we were struggling to get something merged with bors, but where it was clear that the failures weren't related to our PR.

I see there's no clear mention in the docs however, we should probably add it

Roger wilco!