chore: use eslint directly

[legendecas]

tools/eslint-format was not failed on #1613 but failed on the main branch.

Since the diff on applying the eslint is trivial, let's apply eslint on all files.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.35%. Comparing base (cc2b8df) to head (789dacc).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1616   +/-   ##
=======================================
  Coverage   66.35%   66.35%           
=======================================
  Files           3        3           
  Lines        2143     2143           
  Branches      703      703           
=======================================
  Hits         1422     1422           
  Misses        150      150           
  Partials      571      571           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[KevinEady]

Hmm, should the formatting changes have been found in our "format everything" branch #1160 ? Or were there new rules added since 1160 landed (May 2022)?

[legendecas]

#1613 updated the eslint and plugins version. However, tools/eslint-format only checks on JS files changed in a PR -- where #1613 didn't change any JS files. eslint-format does not apply new eslint and plugins on existing files.

Both standard and neostandard have the rule 'object-shorthand' in their new versions.

[KevinEady]

LGTM! 🎉

[mhdawson]

LGTM