Enhancement: replace core implementation with typer/click

MartinPacker / mdpre

mdpre - Markdown Preprocessor

MIT License

8 stars 3 forks source link

Enhancement: replace core implementation with typer/click #2

Closed 42sol-eu closed 1 year ago

42sol-eu commented 4 years ago

(-) first non std lib dependency
(+) readability and reusability

MartinPacker commented 4 years ago

I don’t understand this. Please elaborate.

42sol-eu commented 4 years ago

the mdpre library is stdlib only at the moment and if Pyto would force us to copy the files to its local storage that might be a worthwhile feature.

Onnthe other hand is mdpre packed with arguments that would benefit from automatic documentation and other nice stuff like prompting the user for a mandatory value.

I could creste a mock up to show what I am referring to.

42sol-eu commented 4 years ago

I started a raw frame as a wrapper of both tools. See https://github.com/42sol-eu/cute_gearbox.git.

I would like to start on integrating typer into mdpre first.

Using mdpre or md2pptx from other scripts would make it necessay to make the projects importable (init.py and py-extension on mdpre.

MartinPacker commented 4 years ago

OK. Go with mdpre first - as I want to do line continuation in md2pptx when I get a moment.

And tell me how to test this. (I will before I close the Issue.) This should make it into 0.4.6.

MartinPacker commented 1 year ago

Not going to do this.