Added transform tests. #114

wants to merge 1 commit into


None yet

2 participants

olvlvl commented Jul 23, 2013

Some transform tests.

olvlvl commented Jul 23, 2013

I just realized that you provide unit tests in another repository:

This is really confusing because unit tests are usually available in the main repository. This is especially confusing because there is no mention of that repository in your README. You really should add a TESTING section.

Tests should be included with your software so that it can be tested with a continuous integration platform such as Travis-CI each time modifications are pushed to the repository. It's really handy with pull request too because it can test them before they are merged.

michelf commented Jul 23, 2013

The reason those tests are in a separate repository (and do not depend on PHP Unit) is so they can be used by other Markdown implementations. MDTest still needs some tweaking so it can be used without hassle with the new library-style package in the lib branch, but you're right it should get a mention in the readme file. Sorry for the confusion.

@michelf michelf added a commit that closed this pull request Nov 28, 2013
@michelf Added instructions in the Readme about submitting pull requests, with…
… reference to MDTest.

Fixes #114.
@michelf michelf closed this in fe2c744 Nov 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment