Skip to content

Commit

Permalink
Remove rules about reserved words
Browse files Browse the repository at this point in the history
In 53b4173 we removed the ES5 guide which contains a lot of guidelines
that are no longer very relevant for us. Similarly, some folks have
raised the relevance of these rules about reserved words, given that we
are now living in an age where ES3 support has mostly waned and
transpilers such as Babel are widely adopted and pave over these issues.
This seems like a good opportunity to simplify.

Fixes #61
  • Loading branch information
lencioni committed Aug 3, 2016
1 parent f0b3196 commit bfc842e
Showing 1 changed file with 0 additions and 37 deletions.
37 changes: 0 additions & 37 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -153,43 +153,6 @@ Other Style Guides
const item = {};
```
<a name="objects--reserved-words"></a><a name="3.2"></a>
- [3.2](#objects--reserved-words) If your code will be executed in browsers in script context, don't use [reserved words](http://es5.github.io/#x7.6.1) as keys. It won't work in IE8. [More info](https://github.com/airbnb/javascript/issues/61). It’s OK to use them in ES6 modules and server-side code. jscs: [`disallowIdentifierNames`](http://jscs.info/rule/disallowIdentifierNames)

```javascript
// bad
var superman = {
default: { clark: 'kent' },
private: true,
};
// good
var superman = {
defaults: { clark: 'kent' },
hidden: true,
};
```

<a name="objects--reserved-words-2"></a><a name="3.3"></a>
- [3.3](#objects--reserved-words-2) Use readable synonyms in place of reserved words. jscs: [`disallowIdentifierNames`](http://jscs.info/rule/disallowIdentifierNames)

```javascript
// bad
var superman = {
class: 'alien',
};
// bad
var superman = {
klass: 'alien',
};
// good
var superman = {
type: 'alien',
};
```

<a name="es6-computed-properties"></a><a name="3.4"></a>
- [3.4](#es6-computed-properties) Use computed property names when creating objects with dynamic property names.
Expand Down

0 comments on commit bfc842e

Please sign in to comment.