How do I disable function() to function () fix

[lfender6445]

I love this tool but cant seem to find the JSHint declaration for this -

I'd rather function someFunction() not be changed to function someFunction ()

Thanks

[goatslacker]

that shouldn't be happening. can you add a failing test case?

[jish]

$ cat foo.js
var Animal = function(name) {
  this.name = name;
}
$

$ fixmyjs foo.js
✓ foo.js done.
$

$ cat foo.js
var Animal = function (name) {
  this.name = name;
};$

[goatslacker]

That's expected behavior. According to Crockford's original whitespace rules (which JSHINT inherited from) he mentions that there should be a space after function so this tool follows that advice.

function name() should never be changed to function name () but name = function () is valid.

[lfender6445]

So there is no way to disable?

[goatslacker]

Change your jshint settings to white: false I believe. From what I know white is an unloved feature, jshint is meant to find bugs not complain about styling so this tool is more about fixing bugs rather than beautifying your JS.

I hope to split the two someday, right now, no there is no way to disable certain parts of the style guide -- it's an all or nothing affair :(

Send a pull request if you have any ideas though.

[ariutta]

Is the problem that escodegen adds the spaces? How about using esformatter?

[syrnick]

+1 to the issue. I'd love to turn it off.

[goatslacker]

#110 should solve this issue.