search

amyreese / markdown-pp

Preprocessor for Markdown files to generate a table of contents and other documentation needs
MIT License
309 stars 68 forks source link

Recursive #27

Closed leotrs closed 8 years ago

leotrs commented 8 years ago

Addresses #24. I wanted to add a test before this gets merged but wanted to ask what the best procedure is.

For other tests I have included a couple dummy *.mdpp files inside test/, but to test this one, I would have to create at least one subdirectory inside test/ with one further dummy file. I didn't want to plague the repo with dummy files without asking first.

amyreese commented 8 years ago

Perhaps it would make sense to have a directory tree within the test directory to contain all the test case inputs/outputs. This could then include a further directory for testing the recursive bits. Something like:

leotrs commented 8 years ago

Test branch with proposed datafiles directory was merged.

TODO: write the mentioned test for recursive and then merge this branch.

leotrs commented 8 years ago

@jreese

This should be merged after the incoming PR from my logging branch (which basically changes print for logging.

That one in turn needs to be merged after #31 and #33. I'm closing this one until all of that is done, to reduce clutter.