Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Merge requests
  • !19990

Just use ESLint directly instead of going thru grunt-eslint

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge bypass-grunt-eslint into v4-dev May 31, 2016
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 3

Created by: cvrebert

grunt-eslint buys us essentially nothing over just invoking ESLint directly via an npm script, so this PR updates the build to do just that. Removing the unnecessary layer of indirection will also simplify updating ESLint (#19908 (closed)), since we can set the ESLint version directly.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: bypass-grunt-eslint