query stake-distribution: add --output-{json,text} flags

[smelc]

Changelog

- description: |
    query stake-distribution: add --output-{json,text} flags
# uncomment types applicable to the change:
  type:
  - feature        # introduces a new feature
  - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - improvement    # QoL changes e.g. refactoring
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context

• It appears this is a longstanding request from users, as pointed out by @CarlosLopezDeLara to me
• Contributes to fixing [FR] - Make output of CLI commands consistent. #566
• The design is the same as query leadership-schedule: add --output-[json,text] flag to control format of the output #649, query utxo: add --output-[json,text] flag to control format of the output #611, and [--output-format json|--output-format json] format becomes [--output-yaml|--output-json] #523 to guarantee consistency

Fixes #739

How to trust this PR

I tested this PR's behavior on IntersectMBO/cardano-node#5803. You can see the comparison of the before/after states here: test_743.zip

Checklist

• Commit sequence broadly makes sense and commits have useful messages
• Self-reviewed the diff

[palas]

@smelc, the test_743.zip file you attached seems to only have empty folders

[palas]

Thanks 👍. It seems a completely uncontroversial PR to me

[palas]

Outside of this PR. But I was thinking it would be simpler, nicer, more reusable, and fun to have a

drawTable :: [[String]] -> String

function instead.

[smelc]

@smelc, the test_743.zip file you attached seems to only have empty folders

Urrgh indeed, I had forgotten the -r flag. I reuploaded it 👌