Skip to content

[5.x] [grunt/eslint] fix precommit linting#9532

Merged
spalger merged 1 commit into5.xfrom
jasper/backport/9510/5.x
Dec 16, 2016
Merged

[5.x] [grunt/eslint] fix precommit linting#9532
spalger merged 1 commit into5.xfrom
jasper/backport/9510/5.x

Conversation

@elastic-jasper
Copy link
Contributor

[grunt/eslint] fix precommit linting

Backports PR #9510

Commit 1:
[grunt/eslint] fix precommit linting

  • remove use of minimatch.makeRe() because it does not support the entire glob syntax
  • log a warning whenever a js file is excluded by the lintStagedFiles task
  • eslint globs are relative to the project root, ensure that we check against relative version

Commit 2:
[grunt/eslint] only log warning wtr grunt paths

Backports PR #9510

**Commit 1:**
[grunt/eslint] fix precommit linting

 - remove use of `minimatch.makeRe()` because it does not support the entire glob syntax
 - log a warning whenever a js file is excluded by the `lintStagedFiles` task
 - eslint globs are relative to the project root, ensure that we check against relative version

* Original sha: ca45ae2
* Authored by spalger <spalger@users.noreply.github.com> on 2016-12-15T19:01:21Z

**Commit 2:**
[grunt/eslint] only log warning wtr grunt paths

* Original sha: b152e35
* Authored by spalger <spalger@users.noreply.github.com> on 2016-12-15T21:45:40Z
@elastic-jasper elastic-jasper added the backport This PR is a backport of another PR label Dec 16, 2016
@spalger spalger merged commit aa41e38 into 5.x Dec 16, 2016
@epixa epixa deleted the jasper/backport/9510/5.x branch December 20, 2016 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants