starred/markdownlint

Fork 0

mirror of https://github.com/DavidAnson/markdownlint.git synced 2026-04-26 01:36:03 +03:00

[GH-ISSUE #374] Add a .pre-commit hook #2161

New issue

Closed

opened 2026-03-07 20:05:09 +03:00 by kerem · 7 comments

kerem commented

2026-03-07 20:05:09 +03:00

Owner

Originally created by @JensHeinrich on GitHub (Mar 3, 2021).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/374

The ruby implementation already supplies a pre-commit hook and it would be useful to have one for the npm version too

Originally created by @JensHeinrich on GitHub (Mar 3, 2021). Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/374 The ruby implementation already supplies a [pre-commit](https://pre-commit.com/) hook and it would be useful to have one for the npm version too

kerem

2026-03-07 20:05:09 +03:00

closed this issue
added the
enhancement

fixed in next
labels

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@DavidAnson commented on GitHub (Mar 3, 2021):

Both CLI’s have this already:

@DavidAnson commented on GitHub (Mar 3, 2021): Both CLI’s have this already: - https://github.com/igorshubovych/markdownlint-cli/blob/master/.pre-commit-hooks.yaml - https://github.com/DavidAnson/markdownlint-cli2/blob/main/.pre-commit-hooks.yaml

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@JensHeinrich commented on GitHub (Mar 3, 2021):

Thank you very much.

I did not see it.

Can I add a reference to the CLI section to the example usage section?

@JensHeinrich commented on GitHub (Mar 3, 2021): Thank you very much. I did not see it. Can I add a reference to the CLI section to the example usage section?

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@janosh commented on GitHub (Oct 6, 2021):

I second @JensHeinrich's suggestion to specifically mention pre-commit in the readme. ac8545e did not help me find this.

@janosh commented on GitHub (Oct 6, 2021): I second @JensHeinrich's suggestion to specifically mention pre-commit in the readme. ac8545e did not help me find this.

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@DavidAnson commented on GitHub (Oct 6, 2021):

This library does not work with pre-commit; the CLIs do (and both document support in their README). I'm not sure I want to re-document everything downstream projects support.

@DavidAnson commented on GitHub (Oct 6, 2021): This library does not work with pre-commit; the CLIs do (and both document support in their README). I'm not sure I want to re-document everything downstream projects support.

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@janosh commented on GitHub (Oct 6, 2021):

Fair enough. I noticed you have a file on Prettier in the docs. Perhaps a file for pre-commit could be added as well?

@janosh commented on GitHub (Oct 6, 2021): Fair enough. I noticed you have a file on Prettier in the docs. Perhaps a file for pre-commit could be added as well?

kerem commented

2026-03-07 20:05:10 +03:00

Author

Owner

@DavidAnson commented on GitHub (Oct 6, 2021):

Good idea, I've added a TODO!

@DavidAnson commented on GitHub (Oct 6, 2021): Good idea, I've added a TODO!

kerem commented

2026-03-07 20:05:11 +03:00

Author

Owner

@janosh commented on GitHub (Oct 6, 2021):