Bump eslint from 7.32.0 to 8.30.0 (opensearch-project#346)

* Bump eslint from 7.32.0 to 8.30.0
* Replaced `babel-eslint` with `@babel/eslint-parser`
* Limit linting step to Nodejs 16


Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Theo Truong <theotr@amazon.com>

.eslintrc.json

@@ -7,7 +7,10 @@
         "eslint:recommended",
         "prettier"
     ],
-    "parser": "babel-eslint",
+    "parser": "@babel/eslint-parser",
+    "parserOptions": {
+        "requireConfigFile": false
+    },
     "plugins": [
         "prettier"
     ],

CHANGELOG.md

@@ -12,6 +12,8 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ### Dependencies
 - Bumps `xmlbuilder2` from 2.4.1 to 3.0.2
 - Bumps `minimatch` from 3.0.4 to 3.1.2
+- Bumps `eslint` from 7.32.0 to 8.30.0
+- Replaced `babel-eslint` with `@babel/eslint-parser`
 - Bumps `eslint-plugin-prettier` from 4.0.0 to 4.2.1
 
 ### Dependencies
@@ -36,4 +38,4 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - Fix mutability of connection headers ([#291](https://github.com/opensearch-project/opensearch-js/issues/291))
 
 [2.1]: https://github.com/opensearch-project/opensearch-js/releases/tag/2.1.0
-[Unreleased]: https://github.com/opensearch-project/opensearch-js/compare/2.1...HEAD
+[Unreleased]: https://github.com/opensearch-project/opensearch-js/compare/2.1...HEAD

package.json

@@ -57,15 +57,15 @@
   },
   "devDependencies": {
     "@aws-sdk/types": "^3.160.0",
+    "@babel/eslint-parser": "^7.19.1",
     "@sinonjs/fake-timers": "github:sinonjs/fake-timers#0bfffc1",
     "@types/node": "^15.3.1",
-    "babel-eslint": "^10.1.0",
     "convert-hrtime": "^5.0.0",
     "cross-zip": "^4.0.0",
     "dedent": "^0.7.0",
     "deepmerge": "^4.2.2",
     "dezalgo": "^1.0.3",
-    "eslint": "^7.32.0",
+    "eslint": "^8.30.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-plugin-prettier": "^4.0.0",
     "fast-deep-equal": "^3.1.3",

test/acceptance/observability.test.js

@@ -384,10 +384,10 @@ test('Client name', (t) => {
     client.on('resurrect', (err, meta) => {
       t.error(err);
       t.equal(meta.name, 'opensearch-js');
-      clock.uninstall();
     });
 
     client.info({}, { id: 'custom' }, noop);
+    clock.uninstall();
   });
 
   t.test('Resurrect should have the client name configured (child client)', (t) => {
@@ -412,10 +412,10 @@ test('Client name', (t) => {
     client.on('resurrect', (err, meta) => {
       t.error(err);
       t.equal(meta.name, 'child-client');
-      clock.uninstall();
     });
 
     child.info({}, { id: 'custom' }, noop);
+    clock.uninstall();
   });
 
   t.end();