search

jquery / sizzle

A sizzlin' hot selector engine.
https://sizzlejs.com
Other
6.29k stars 951 forks source link

Fix 438 use eslint #440

Closed JuanMaRuiz closed 5 years ago

JuanMaRuiz commented 5 years ago

Build: Configure eslint as lint tool

As discussed in the opened issue jscs is deprecated and jQuery is using eslint as linter tool. I have configured eslint as is in the jQuery project. I have had to reconfigure to skip the rule "no-nested-ternary"

  "rules": {
    "no-nested-ternary": "warn"
  }

because the style of coding of sizzle is not the same of the jQuery and the build increase in weight if I try to replace the ternaries with ifs.

Fixes #438 Ref #438

jsf-clabot commented 5 years ago

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: JuanMaRuiz
:x: JuanMa Ruiz

JuanMa Ruiz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.