Add ECMAScript 6 features

[nzakas]

This is a rollup that will let us keep track of ES6 progress.

• let declarations
• const declarations
• Regular expression u flag
• Regular expression y flag
• Binary literals
• New octal literals
• Arrow functions
• Default parameters
• Rest parameters
• Spread operator
• super references
• Classes
• for-of loops
• Generators
• Object literal property shorthand
• Object literal method shorthand
• Object literal computed properties
• Template strings
• Destructuring
• Unicode code point escapes
• Allow duplicate object literal properties (except __proto__)
• Modules

[nzakas]

The let and const changes are in. Next up: regular expression flags.

[nzakas]

In order to make the regular expression flags work, I needed to add a regex property to both AST nodes and literal nodes to coincide with Esprima Harmony. I've got this working, but need to update the AST tests for the various third-party libraries that Esprima uses to validate AST output.

[FredKSchott]

@nzakas which of these would you reccomend to get started on?

[nzakas]

I think allowing duplicate properties in strict mode is the easiest to get started with. I'd call that objectLiteralDuplicateProperties.

After that, Unicode code point escapes (see https://leanpub.com/understandinges6/read#leanpub-auto-better-unicode-support)

[nzakas]

I'm currently working on generators.

[FredKSchott]

I'm starting work on Unicode code point escapes

[nzakas]

Generators and code point escapes have been merged in.

[nzakas]

Next priority is template literals. @FredKSchott interested in taking that?

[FredKSchott]

definitely, I'll tackle that next (away for the long weekend but I'll take a look next week)

They're supported in V8 now, right? Makes sense to do them next

[nzakas]

Yup! And also in io.js now.

I'm planning on attacking arrow functions when I have some energy.

[endel]

facebook/esprima supports classes.

[nzakas]

Esprima classes support: jquery/esprima#1061

[xjamundx]

Well that makes things easier :)

[nzakas]

Working on classes now.

[xjamundx]

👏

[FredKSchott]

you guys are killing it 👏 👏 👏

[nzakas]

Ran into a bit of trouble with classes. The Harmony branch is different that Esprima's master now (master being the most correct). Will probably take me a bit to work through it unless someone else wants to give it a shot.

[xjamundx]

I'm still trying to grok the destructuring defaults spec. Once I'm done with that I'll try to throw my hat into either classes or modules. Whatever's left....

[niels4]

Fast work on getting these es6 features implemented!

I noticed an issue with rest parameters though, they don't seem to get added to the scope's variable list. I noticed this when trying to use eslint to validate a function with rest parameters.

var func = function (first, ...rest) {
  return rest;
};

I am getting a 'rest' is not defined (no-undef) error. The underlying cause seems to be that when the rule's function looks at context.getScope().variables, a variable named 'rest' is not in the list.

see eslint/eslint#1870 to fully replicate

[xjamundx]

Cool just saw eslint/eslint#1870.

[nzakas]

As I mentioned there, this is an issue with escope, not espree.

[nzakas]

Don't look now, modules is the last feature!

[xjamundx]

This is great news!! Well done!

[fczuardi]

should we file a separate bug just for the modules support?

In this related bug I posted an example of how espree differs from esprima when used with escope eslint/eslint#1978 (comment)

[nzakas]

Modules are already properly supported.