Markdown-toc fails if there are multiple .md files

[nymous]

After #4 landed the precommit fails if multiple Markdown files are present, because markdown-toc doesn't support multiple input files on its CLI. This is sad because it was the point of that PR 😅

This issue has been reported by Yajo itself here pre-commit/pre-commit#1374

My current workaround is to use the pre-commit offered by DocToc (as mentioned in the pre-commit issue).

Note: This issue is more of a heads-up for others who would run into the same problem because it would need to be fixed in the markdown-toc project directly.

[Lucas-C]

I opened jonschlinkert/markdown-toc#162 to suggest the evolution in markdown-toc

[nymous]

Awesome! I was checking in the upstream issues to see if someone already asked for this before creating a new one but you beat me to it ^^

[beliaev-maksim]

would be helpful to have it

now have to use multiple hooks for each file:

  - repo: https://github.com/Lucas-C/pre-commit-hooks-nodejs
    rev: v1.1.1
    hooks:
      - id: markdown-toc
        name: README.md-toc
        files: ^README\.md$
      - id: markdown-toc
        name: configuration.md-toc
        files: .*/configuration\.md$
      - id: markdown-toc
        name: CONTRIBUTE.md-toc
        files: .*/CONTRIBUTE\.md$

[Lucas-C]

This depend on jonschlinkert/markdown-toc#162
You may want to drop a comment there

[beliaev-maksim]

@Lucas-C
we got a green light for update of CLI

would you like to create a PR for this?

[drjasonharrison-vp-eio]

What do we need to fix this?

[Lucas-C]

Have this PR merged: jonschlinkert/markdown-toc#172

[drjasonharrison]

How much longer to wait until we just switch to your fork of the repo?
Or maybe @Lucas-C would be willing to take on the latest leading fork? https://github.com/Lucas-C/markdown-toc

[beliaev-maksim]

+1 on switching the fork until they merge

[Lucas-C]

I released a v1.1.2 version that uses my GitHub fork of the markdown-toc lib

This issue should be solved now

[drjasonharrison]

Thanks @Lucas-C!