Replacing eslint with standard cli

[etroynov]

Hi,

While working on one of the plugins for koa which is used in our company I needed to study some points in the Koa source code, during the installation of dependencies I noticed deprecating warnings.

In the course of studying the project's dependencies I found:

• eslint-plugin-node - the plugin has not been updated for the last 4 years and for this reason a fork of eslint-plugin-n was created which is recommended for use;
• eslint-plugin-standard - marked as deprecated and the official standard website recommends using their cli utility;
• .eslintrc.yml - does not have any special settings and uses only:

  extends: standard

why not just update eslint?

I can see several reasons why the official utility is better than the eslint custom config ( in my opinion ofc ):

• fewer dependencies - With eslint we have to monitor and update the list of dependencies:

  "eslint": "^7.32.0",
  "eslint-config-standard": "^16.0.3",
  "eslint-plugin-import": "^2.18.2",
  "eslint-plugin-node": "^11.1.0",
  "eslint-plugin-promise": "^5.1.0",
  "eslint-plugin-standard": "^5.0.0",

  With standartd we have only two dependecies and one of it is optional:

  "snazzy": "^9.0.0", // just for pretty output
  "standard": "^17.1.0",

• deprecated dependencies - If, for example, one of the dependencies becomes unsupported, we will need to find a replacement for it, which will also need to be monitored, so it seems to me that if we can avoid this, we should do it.
• official way - If a tool has an official cli that is maintained and updated by their team, why not just use it?

Checklist

• [x] I have ensured my pull request is not behind the main or master branch of the original repository.
• [x] I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
• [x] I have written a commit message that passes commitlint linting.
• [x] I have ensured that my code changes pass linting tests.
• [x] I have ensured that my code changes pass unit tests.
• [x] I have described my pull request and the reasons for code changes along with context if necessary.

[socket-security[bot]]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Packages	Version	New capabilities	Transitives	Size	Publisher
snazzy	9.0.0	None	+7	247 kB	feross
standard	17.1.0	None	+143	16.4 MB	voxpelli

🚮 Removed packages: eslint@7.32.0, eslint-config-standard@16.0.3, eslint-plugin-import@2.26.0, eslint-plugin-node@11.1.0, eslint-plugin-promise@5.2.0, eslint-plugin-standard@5.0.0

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5ba533f) 98.66% compared to head (4db592c) 98.66%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #1796 +/- ## ======================================= Coverage 98.66% 98.66% ======================================= Files 5 5 Lines 523 523 Branches 146 146 ======================================= Hits 516 516 Misses 7 7 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[etroynov]

@dead-horse, @fengmk2

Can you take a look?

[etroynov]

@fengmk2 thx a lot :)