Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add parameter (value and default) information to rule documentation. #76

Merged
merged 1 commit into from
Mar 19, 2015
Merged

Add parameter (value and default) information to rule documentation. #76

merged 1 commit into from
Mar 19, 2015

Conversation

DavidAnson
Copy link
Contributor

Seemed useful to document this outside the code.

@mivok
Copy link
Collaborator

mivok commented Mar 19, 2015

It fails CI because I have a test in there to make sure the docs match the actual rules - it shouldn't be too hard to make the test ignore anything in parentheses and then it should pass. Once I've done that this should be OK to merge.

@DavidAnson
Copy link
Contributor Author

Thanks for catching that, Mark! I dashed this PR off on the bus without a means to run the tests. I won't do that again.

@mivok mivok merged commit 66f84a3 into markdownlint:master Mar 19, 2015
@DavidAnson DavidAnson deleted the paramdoc branch March 19, 2015 17:06
@DavidAnson
Copy link
Contributor Author

Awesome - thanks for the assist! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants