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

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

Merged
merged 1 commit into from
Dec 16, 2016
Merged

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 <[email protected]> on 2016-12-15T19:01:21Z

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

* Original sha: b152e35
* Authored by spalger <[email protected]> on 2016-12-15T21:45:40Z
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants