Skip to content

Conversation

@akx
Copy link
Contributor

@akx akx commented May 15, 2025

Just some papercuts I noticed while trying out the CLI:

  • args.run() is an unfortunate place to do argument validation; can just as well do it while parsing.
  • I messed up my bbox the first time around and got an unhelpful error. No more!
  • It's useful to be able to always choose to save the Overpass data to a file, instead of only when happening to use --debug, so that's now a thing (--save-json-file).

@akx akx force-pushed the retrieve-improvements branch from 07f9751 to 8dc48a5 Compare May 15, 2025 12:23
@akx akx force-pushed the retrieve-improvements branch from 8dc48a5 to fdc5156 Compare May 15, 2025 12:42
@louis-e
Copy link
Owner

louis-e commented May 15, 2025

Great improvements, thanks a lot for your contribution! :)

@louis-e louis-e merged commit 400004e into louis-e:main May 15, 2025
3 checks passed
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.

2 participants