Skip to content

Extends eslint-config-airbnb-base with OpenTable's internal preferences.

Notifications You must be signed in to change notification settings

opentable/eslint-config-opentable

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

40 Commits
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 

Repository files navigation

OpenTable

eslint-config-opentable Build Status

This package provides OpenTable's .eslintrc as an extensible shared config. We extend eslint-config-airbnb-base.

Usage

The export lints ES6/2015+. It requires eslint and eslint-plugin-import.

  1. npm install --save-dev eslint-config-opentable eslint
  2. add "extends": "opentable" to your .eslintrc

See Airbnb's Javascript styleguide and the ESlint config docs for more information.

Dependencies

Note that ESLint 3.0+, which is a dependency of this config, requires Node 4+. For support in Node <4, use eslint-config-opentable version 4.x.

Differences with AirBnB

There are a few minor differences between this config and AirBnB's. Links are to AirBnB's styleguide.

  • 18.12 Max line length: 100 - Increased to 120

    Why? We just wanted to allow longer lines.

  • 19.2 Comma dangle - No

    Why? We found the extra comma distracting.

  • 22.3 Use PascalCase only when naming constructors or classes. - Yes, but one exception

    Why? We added $.Deferred as a common exception, since it should not be used with the new keyword.

Maintainers

This repo doesn't change too often, so issues or requests may not be noticed quickly. Please contact the maintainers directly to call an issue to their attention.