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

Fix more markdown #1603

Merged
merged 2 commits into from
Nov 1, 2017
Merged

Fix more markdown #1603

merged 2 commits into from
Nov 1, 2017

Conversation

mroderick
Copy link
Member

Immediately after merging #1600 I discovered a trailing quote in one of the headings. During fixing that, I also discovered that the pre commit hook didn't run any validations for the changed markdown files.

This PR fixes both those issues.

To verify

  1. Check out this branch
  2. Add a Markdown violation to one of the files in docs/. <a href="asdf">asdf</a> works well for me.
  3. git add docs/
  4. git commit
  5. Observe that markdownlint aborts the commit and spits out an error message

The minimatch pattern in `package.json` was not matching anything at
all. Currently, I can't figure out how to write a minimatch supported
pattern that will "include any file from docs, but not `changelog.md`".

Luckily, `changelog.md` should only be updated by `postversion.sh`,
which doesn't have any human interaction. I've updated `postversion.sh`
to bypass the pre commit hook.
@fatso83 fatso83 merged commit d49180d into sinonjs:master Nov 1, 2017
@mroderick mroderick deleted the fix-more-markdown branch November 1, 2017 10:01
@mroderick
Copy link
Member Author

Thanks

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