func-names with ES6 classes

[Marsup]

ESLint seems to misunderstand ES6 classes definitions.

test.js :

class A {
  constructor() {}
  foo() {}
}

.eslintrc

ecmaFeatures:
  classes: true
rules:
  func-names: 2

Errors :

test.js
  2:13  error  Missing function expression name  func-names
  3:5   error  Missing function expression name  func-names

I can make probably make a patch but I don't know if the problem is in the parser or in ESLint.

[nzakas]

The problem is in the rule. Concise methods are represented by FunctionExpression, so we'd need to check to see if the parent is MethodDefinition and, if so, ignore it.

Feel free to take a stab at it, please take moment to read our contribution guidelines before you start: http://eslint.org/docs/developer-guide/contributing.html

[Marsup]

I'm working on it, almost done. Is it normal that a setter missing its argument provides 0:0 error Message reference must be in range as error ? It took me a while to notice that's what was missing.

[nzakas]

I'm unfamiliar with that error message. Please file an issue with Espree describing the code you're using and the error you're seeing.