Rollup: Allow definition of jQuery style

In order to make sure we have the correct style rules, I’d like to use the jQuery style guide as a reference. If we can implement the jQuery style guide in ESLint, then we will have a good set of style rules as a basis.

• Identation check (#1022)
• No trailing whitespace
• Unary operators must not have spaces (space-unary-ops)
• Ternary statements must have spaces before and after both ? and : (space-infix-ops)
• Maximum line length (max-len)
• Require braces for control statements (curly)
• Require multiple lines for control statements (curly)
• Commas cannot have a preceding space (#628)
• Semicolons cannot have a preceding space (no-space-before-semi)
• Semicolons only at end of line (semi)
• The : in property names should have no preceding space (#1280)
• No filler spaces in empty constructs like [], {}
• New line at end of file
• If entire file is wrapped in closure, the body should not be indented
• Spaces inside of parens (#627)
• Spaces inside of square brackets (#631)
• Line breaks must occur after an operator (#1405)
• Always use semicolons (semi)
• Use camel case (camelcase)
• Allow non-capital constructor names
• Align case with switch

<bountysource-plugin>

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource. </bountysource-plugin>

[mysticatea]

There is an workaround.

$('#somebutton').click(/* @this HTMLElement */ function(){
    var pid = $(this).attr('pid');
    doAction(pid);
});

[ndaidong]

How about “no-invalid-this”. While using jQuery, it’s normal to write:

  $('#somebutton').click(function(){
    var pid = $(this).attr('pid');
    doAction(pid);
  });

With “no-invalid-this” enabled, that will become an error.