no-invalid-this triggers an error on a function expression constructor

What version of ESLint are you using? v3.2.2 What parser (default, Babel-ESLint, etc.) are you using? default Please show your full configuration:

https://gist.github.com/swang/ab31cd9d11593568a6ada2d8badeb92c

What did you do? Please include the actual source code causing the issue. this is smallest reproduced code that generates this error.

'use strict';

var TestClass;

TestClass = function() {
    this.x = 1;
};

What did you expect to happen? no-invalid-this should not trigger since TestClass begins with an uppercase letter.

What actually happened? Please include the actual, raw output from ESLint. I get an error from the no-invalid-this rule

[mysticatea]

Yup. Currently, the rule checks whether the destination of assignments is a property or not. But it does not check whether the destination of assignments is an ES5 style constructor (i.e. starts with upper case) or not.

[mysticatea]

@michaelficarra At least, we can use this to avoid conflict with (func-style: expression & func-names: never).