pnsaevik / effluent

MIT License
3 stars 1 forks source link

CLI #18

Closed castelao closed 1 year ago

castelao commented 1 year ago

There is nothing wrong with how you did it with __main__. I usually define an external script, and take advantage of custom controls with argparse or click. Here an example using click.

Just a suggestion, feel free to ignore it.

pnsaevik commented 1 year ago

Thanks for the suggestion! I tend to avoid using external scripts, to avoid polluting the CLI namespace. But for this package, I think the extra user-friendliness is worth it. I'll draft a fix and merge into main. There will be minor changes to the documentation as well, which you can inspect at https://effluent.readthedocs.io/en/main/