Forces ecma6 about eslint-config-xo HOT 9 CLOSED

It has almost always done that. I like it this way as it will parse anything without having to set an option. You can easily just override if you don't agree, though. That's the beauty of shareable configs :)

from eslint-config-xo.

It does not override, though. It will take the the plugin version...the problem is setting the parserOptions explicitly. If that's not set, all works fine. (This was added recently, 83aad21)

from eslint-config-xo.

Yes, but it already had the es6 environment variable set, which triggered ES6 parsing implicitly. 83aad21#diff-168726dbe96b3ce427e7fedce31bb0bcL11 So nothing new here.

from eslint-config-xo.

The es6 environment variable gets overriden, however, parserOptions do not get overriden. So this is new and a breaking change for us :(

from eslint-config-xo.

If you're saying you can not do the following, it sounds like a bug in ESLint and should be reported there:

{
    extends: 'xo',
    parserOptions: {
        ecmaVersion: 5
    }
}

from eslint-config-xo.

Plugins override the parser options (I asked the es team); I will put in a feature request (they asked me to see if xo team will change :P)

from eslint-config-xo.

@sjswoboda I know, but this is not a plugin. It's a shareable config.

from eslint-config-xo.

Sorry, I meant the shareable config is overriding the parser options. (no plugin here) I was under the impression that this is an eslint config -- in which case anyone using this config (esling-config-google, etc.) are all now getting "upgraded" to ecma6 because eslint does not support (currently) overriding parserOptions.

from eslint-config-xo.

eslint does not support (currently) overriding parserOptions.

And if so, is a ESLint problem, and should be opened there.

from eslint-config-xo.