Improvement Thoughts

[chrisjsewell]

As discussed in executablebooks/markdown-it-py#10, great work!

A few thoughts off the cuff:

• use click for CLI, which can also read configuration from a config file (see https://github.com/executablebooks/scss-compile for an example)
• Plugin system for code block formatting (e.g. black for python blocks)
• Add tox configuration, for test automation

[hukkin]

Thanks for your thoughts!

• use click for CLI, which can also read configuration from a config file (see https://github.com/executablebooks/scss-compile for an example)

I think this is a great idea and should be implemented if we end up having configuration. Currently there is zero configuration, so I'd be hesitant to add a new dependency if argparse can still get the job done just fine.

• Plugin system for code block formatting (e.g. black for python blocks)

I really like this. I made a separate issue #12

• Add tox configuration, for test automation

If there is a problem that this would solve, I don't see why not. Currently though we have no sys.version_info checks (meaning all Python versions do pretty much the same stuff), and no dependencies where we would need to test support for more than one major version, so while developing I've found simply running pytest sufficient. CI runs tests on all OS and py versions though.

[chrisjsewell]

If there is a problem that this would solve, I don't see why not

Mainly just for me lol; when jumping around between lots of repositories, its just really helpful to be able to quickly run tox, rather than manually setting-up/initializing new dev environments every time

[chrisjsewell]

Whaay thanks @hukkinj1, welcome to EBP 🥳 (cc @choldgraf @mmcky @AakashGfude).
Think I'll just delete the whole package now 😆

[hukkin]

Think I'll just delete the whole package now

[chrisjsewell]

even if you found me, you would still have to wait a few weeks in quarantine 😆

[hukkin]

Oh man, add another two weeks to that after the flight back. It might just not be worth it. You win.

To conclude this ticket: I think there's no immediate action items on this issue, so closing. A tox PR is welcome if someone finds it helpful. Click shouldn't IMO be rushed until it solves a problem (e.g. reading a configuration file).