Allow glob exclusions

Some projects commit build (dist) files to the repository. Usually, the syntax of these generated files is out of the scope of linters. But ATM all files that matches the glob pattern will run against linters.

Ideally we want *-staged tasks to be configured the same way as normal linter tasks.

But this might require a major rewrite and switch to the Node from CLI.

[jharris4]

Ouch, this burned me too. I was trying to use !(node_modules|dist|lib|es)**/*.js for my pattern. And the minimatch docs say:

If the pattern starts with a ! character, then it is negated. Set the nonegate flag to suppress this behavior, and treat leading ! characters normally. This is perhaps relevant if you wish to start the pattern with a negative extglob pattern like !(a|B). Multiple ! characters at the start of a pattern will negate the pattern multiple times.

I’m trying to see if there’s a way to work around this with a different pattern, but otherwise I’ll have to think about making a PR to support passing the nonegate flag to minimatch somehow…

[nikhil618]

Hey there @okonet @jharris4 @davidhund ,

I was naively trying to do something like:

"lint-staged": {
    "src/**/*.js server/src/**/*.js, !server/src/models/**/*.js ": [
      "prettier --write",
      "git add"
    ]
  }

but it doesn’t seem to be working. Do I need to do something else to get the functionality I need?

Maybe you could use https://yarnpkg.com/en/package/multimatch instead of minimatch to support what I’m trying to do?

Thanks!

For anyone who stumbles upon this issue like me it now supports glob match but uses micromatch now read this post I added for the fix. https://stackoverflow.com/a/57981925/1221731