New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support default parameters in ES6 and moz #1195

Closed
MikeRatcliffe opened this Issue Jul 24, 2013 · 0 comments

Comments

Projects
None yet
1 participant
@MikeRatcliffe
Copy link

MikeRatcliffe commented Jul 24, 2013

Default parameters are currently not supported and using them completely invalidates the linting from that line onwards. For Firefox devs this means that it is often only possible to lint the first few lines of a file.

Try linting this:

function Doit() {
}

Doit.prototype = {
  _someProperty: null,

  test: function(num, num2=1) {
    return num === num2;
  },

  get someProperty() {
    return this._someProperty;
  },
};

It would be awesome if support could be added for this feature in ES6 and moz modes.

@valueof valueof closed this in cac4daa Aug 1, 2013

jugglinmike added a commit to jugglinmike/jshint that referenced this issue Oct 21, 2014

Fixed jshint#1195: Add support for ES6 default parameters.
Also fixed jshint#1208.

Signed-off-by: Anton Kovalyov <anton@kovalyov.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment