Skip to content

Allow reserved words as keys in ES6 module context - #332

Merged
hshoff merged 1 commit into
airbnb:masterfrom
tomek-he-him:patch-5
Jun 25, 2015
Merged

Allow reserved words as keys in ES6 module context#332
hshoff merged 1 commit into
airbnb:masterfrom
tomek-he-him:patch-5

Conversation

@tomek-he-him

Copy link
Copy Markdown

<script type="module"> and node code imply full support of ES5.

Code transpiled by babel is also perfecly safe:

$ cat <<––– | babel
const superman = {
  default: { clark: 'kent' },
  private: true
};
–––
"use strict";

var superman = {
   "default": { clark: "kent" },
   "private": true
};

@goatslacker

Copy link
Copy Markdown
Collaborator

+1 for following this change. @hshoff consensus?

@christophehurpeau

Copy link
Copy Markdown
Contributor

👍

@justjake

Copy link
Copy Markdown
Collaborator

LGTM. Rebase this and I'll merge it.

`<script type="module">` and *node*/*iojs* code imply full support of ES5.

Code transpiled by *babel* is also perfecly safe:

```js
$ cat <<––– | babel
const superman = {
  default: { clark: 'kent' },
  private: true
};
–––
"use strict";

var superman = {
   "default": { clark: "kent" },
   "private": true
};
```
@tomek-he-him

Copy link
Copy Markdown
Author

🏁

@hshoff

hshoff commented Jun 25, 2015

Copy link
Copy Markdown
Member

🏆

hshoff added a commit that referenced this pull request Jun 25, 2015
Allow reserved words as keys in ES6 module context
@hshoff
hshoff merged commit 10f6d6a into airbnb:master Jun 25, 2015
@tomek-he-him
tomek-he-him deleted the patch-5 branch June 25, 2015 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants