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
  • !23591

Tweak ESLint rules

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged XhmikosR requested to merge v4-dev-xmr-tweak-eslint-rules into v4-dev Aug 21, 2017
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 21
  • removed a few enabled rules which we are already using the default values
  • enabled a few for strictness

@Johann-S: I know we disagreed last time on undefined but I still believe we should enable this. That being said, the other changes are good too, so I can revert that part if you insist.

We should also discuss which other rules we might enable. I know for sure I'd like semicolons, but it's a personal matter of taste plus this would conflict with your other PRs so it's something we should do in the future, if ever.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: v4-dev-xmr-tweak-eslint-rules