contribute.jquery.org icon indicating copy to clipboard operation
contribute.jquery.org copied to clipboard

The JavaScript Style Guide incorrectly says all projects use JSHint

Open mgol opened this issue 9 years ago • 3 comments

The JavaScript Style Guide incorrectly says:

Every jQuery project has a Grunt task for linting all JavaScript files: grunt jshint

and then describes the JSHint config. jQuery Core uses ESLint for linting and doesn't have .jshintrc or the jshint Grunt task.

mgol avatar Jul 26 '16 11:07 mgol

Hi, I would like to help with this issue. How can I help with it? Where can I find the documentation about eslint configuration? All the links in this page are broken

Best regards.

JuanMaRuiz avatar Jan 16 '19 18:01 JuanMaRuiz

@JuanMaRuiz at the point this documentation was written, we envisioned that all the projects would share a common setup as far as specific tools and style. However, as we've grown it seems less like a good idea.

I think it might be better to have a general statement here that says something to the effect of, "We recommend that each project create code a code style that the team agrees represents best practice, and enforce those practices with tools like ESLint and Prettier. However, each project is free to use their tools of choice to accomplish this." I suspect this could replace nearly the whole page. What does everyone else think about doing that?

dmethvin avatar Feb 17 '19 21:02 dmethvin

@dmethvin that sounds reasonable to me

kswedberg avatar Feb 17 '19 21:02 kswedberg