"Use of future reserved word in strict mode" false positive with eslint-env es6

ESLint version: 0.17.1 (and 67936e0d571c613ac54f4097b4235e6411af0b79 gives the same results)

foo.js:

/*eslint-env es6 */
let x = 42;

.eslintrc:

{
    "ecmaFeatures": { "modules": true }
}

The false positive:

$ ./node_modules/.bin/eslint foo.js 

foo.js
  2:1  error  Use of future reserved word in strict mode

✖ 1 problem (1 error, 0 warnings)

[nzakas]

Most likely we’re not apply ecmaFeatures for inline environments correctly. As a workaround, you can try adding env: { es6: true } to your config file.

[nzakas]

@ilyavolodin we can’t leave this. Either we remove the es6 environment or we figure out a way to make this work.

We could, in theory, just search for /*eslint-env and see if there’s es6:true before parsing.