search

noisebridge / MediaBridge

2 stars 3 forks source link

Consider using typer #30

Open audiodude opened 2 weeks ago

audiodude commented 2 weeks ago

Clearly the test assignment works. But the fact we felt a (helpful!) comment was needed suggests that maybe we're working too hard. Be lazy! Let typer sweat the details about cracking argv. Add a prod: bool parameter to the process_data() signature. Then you will get --help "for free". And no, you certainly don't need to do that in this PR, you might defer that to a subsequent PR.

Originally posted by @jhanley634 in https://github.com/noisebridge/MediaBridge/issues/17#issuecomment-2438606052

skyfenton commented 2 weeks ago

Def support this, including for future tools. Is it too early to consolidate the entry points to these scripts in one command line tool with common constants, etc.? Maybe more of a documentation issue, but I still don't know what some files are for yet.

audiodude commented 1 week ago

The current file structure is based on guesses before we had any code. Most of it is obsolete and misplaced, but we can tackle that once we have more code and a better idea of what we're building.