Object "spread" properties (stage 2 proposal) not supported

[CoericK]

I got something like this

  args: {
    ...connectionArgs,
  },

And jshint its complaining about
xpected '}' to match '{' from line 167 and instead saw '...'.

What should i add to my .jshintrcfile to allow destructing?

[jugglinmike]

JSHint does not currently support this proposed language feature. It recently reached "Stage 2" in the standardization process, though, so we are interested in adding support in a future version.

[CoericK]

Oh thank you @jugglinmike, any idea about how can i make jshint ignore it at least?

[jugglinmike]

Sure, you'll want to use the in-line directives jshint ignore:start and jshint ignore:end:

  args: {
+   // jshint ignore:start
    ...connectionArgs,
+   // jshint ignore:end
  },

[jugglinmike]

I've updated the issue title to more accurately describe the feature request.

[kevlarr]

Would be great to have JSHint support for this, since it's now Stage 3

[derwaldgeist]

+1 for supporting this. It's already widely used in React/Redux

[saidkholov]

+1. Is anyone working on this?

[israbbani]

+1 for getting this done. Thanks guys!

[dcantatore]

+1 would enjoy this very much

[elmargasimov]

+1

[leonardolessa]

+1

[jugglinmike]

Support for this language feature has been merged to the v2.10.0 feature branch (via gh-3150), so we expect it to be included in the next minor release of JSHint. I'll label this issue as "has PR" to better communicate this status.

[renphi]

+1 can't wait for v2.10.0 to be released. Good job! Is there any timeline for v2.10.0?

[jaredatron]

When running jshint@2.9.5

When I have both

{
  "esnext": true,
  "esversion": 6,
}

I get the following error:

Incompatible values for the 'esversion' and 'esnext' linting options. (0% scanned).

[ilyakamens]

Same here re: Incompatible values for the 'esversion' and 'esnext' linting options. (0% scanned).

What's the timeline for 2.10.0?

[chaddjohnson]

I also get Incompatible values for the 'esversion' and 'esnext' linting options even after installing directly from the v2.10.0 branch on GitHub. The error goes away when I omit esversion altogether. Here is a snippet from my .jshintrc, now:

"esnext": true,
"unstable": {
  "objspreadrest": true
},

[caitp]

The idea of that error is really "don't use both 'esversion' and 'esnext' (or 'es3', or 'es5'), afaiu.

The text could be clearer, but the goal of the error seems to be "emit warnings so that eventually we can remove these options and people just use 'esversion').

@jugglinmike do you think we should just make it a non-fatal error instead, or otherwise allow explicit 'esversion' to override the legacy version options?

[jugglinmike]

@chaddjohnson That branch is available to promote transparency and to enable collaboration. You are of course welcome to run any code in this project, but please understand that we do not endorse the use of unreleased code because we do not have the bandwidth to support it.

@caitp I'd prefer to disallow ambiguity rather than attempt to interpret it. I'll share more detail in a response to your pull request.

[NatoBoram]

#2991 (comment)

[jshint] Incompatible values for the 'esversion' and 'esnext' linting options. (0% scanned). (E059)

#2991 (comment)

[jshint] Bad option: 'unstable'. (E001)

@jugglinmike Then do you mean to say that it's impossible right now to use jshint?

[jugglinmike]

Not at all, @NatoBoram. I'm discouraging the use of unpublished versions of the project.

[rip3rs]

Hmmm as anyone found a solution for the:

Incompatible values for the 'esversion' and 'esnext' linting options. (0% scanned). (E059)

[ainthek]

any resolution of this ? when will jshint support LATEST syntaxes ?

[stefanuddenberg]

Any word on this? I really appreciate this tool, but needing to litter my scripts with "ignore" comments is the worst.

[mleonhard]

Would someone have time to take a look at this? I'm running 2.9.7 which is the latest version of jshint, inside Atom editor, and my React Native app code is littered with syntax errors about the spread operator in object literals. Setting "esnext":true in .jshintrc doesn't help. The tool refuses to work when both "esnext":true and "esversion":6 are set.

If anyone has a workaround that works right now, would you please post it?

[jugglinmike]

Support for this feature will be available in JSHint version 2.10.0 which we expect to release early next
week. I'll mark this issue as "resolved" once that's published.

[jugglinmike]

We just released support for object rest/spread in JSHint version 2.10.0.