search

uber / h3

Hexagonal hierarchical geospatial indexing system
https://h3geo.org
Apache License 2.0
4.8k stars 457 forks source link

DRY the CLI #826

Closed dfellis closed 4 months ago

dfellis commented 5 months ago

Based on the discussion with @nrabinowitz in the prior CLI PR. This PR removes duplicate help text and argument types, and turns the arg length trick (and all of the "show subcommand-specific help text" code) into a macro, deleting about twice as much text as it adds.

PS. DRY the CLI does not rhyme.

coveralls commented 5 months ago

Coverage Status

coverage: 98.824% (-0.003%) from 98.827% when pulling ea7489f5ef218baadd2807498c5cc17512e515f5 on dry-the-cli into b49e72e6b3c6be7130c86236340d1b62a31496d2 on master.