@gitlab/eslint-plugin 12.1.0 → 12.3.0

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+# [12.3.0](https://gitlab.com/gitlab-org/frontend/eslint-plugin/compare/v12.2.1...v12.3.0) (2022-05-20)
+
+
+### Features
+
+* Enable no-dynamic-require rule to prevent import expressions ([c260068](https://gitlab.com/gitlab-org/frontend/eslint-plugin/commit/c260068542710f5a7417603d4ea0eddefdbb88a4))
+
+## [12.2.1](https://gitlab.com/gitlab-org/frontend/eslint-plugin/compare/v12.2.0...v12.2.1) (2022-05-20)
+
+
+### Bug Fixes
+
+* Loosen new inherited rules ([08cacf8](https://gitlab.com/gitlab-org/frontend/eslint-plugin/commit/08cacf812534b830f9aa90b9f7c6b90ea2698a56)), closes [/github.com/airbnb/javascript/blob/366bfa66380c08304101c6add46355696e90b348/packages/eslint-config-airbnb-base/rules/es6.js#L67-L68](https://gitlab.com//github.com/airbnb/javascript/blob/366bfa66380c08304101c6add46355696e90b348/packages/eslint-config-airbnb-base/rules/es6.js/issues/L67-L68)
+
+# [12.2.0](https://gitlab.com/gitlab-org/frontend/eslint-plugin/compare/v12.1.0...v12.2.0) (2022-04-21)
+
+
+### Features
+
+* Update eslint-config-airbnb-base to be eslint@8 compatible ([d006b5b](https://gitlab.com/gitlab-org/frontend/eslint-plugin/commit/d006b5b9bdcbb70d6f8ddf0a89c7c37d59dd6507))
+* Update eslint-plugin-import to have eslint@8 support ([c03cffd](https://gitlab.com/gitlab-org/frontend/eslint-plugin/commit/c03cffd795fe516cb8c9d3d4f152b53c2de91d57))
+
 # [12.1.0](https://gitlab.com/gitlab-org/frontend/eslint-plugin/compare/v12.0.1...v12.1.0) (2022-04-07)
 
 

package/lib/configs/default.js

@@ -1,4 +1,4 @@
-const [, ...restrictedGlobals] = require('eslint-config-airbnb-base/rules/variables.js').rules[
+const [, ...restrictedGlobals] = require('eslint-config-airbnb-base/rules/variables').rules[
   'no-restricted-globals'
 ];
 
@@ -26,10 +26,12 @@ module.exports = {
         ignoreDestructuring: true,
       },
     ],
+    'default-param-last': 'off',
     'unicorn/filename-case': ['error', { case: 'snakeCase' }],
     'import/prefer-default-export': 'off',
     'import/no-default-export': 'error',
     'import/no-deprecated': 'error',
+    'import/no-dynamic-require': ['error', { esmodule: true }],
     'no-implicit-coercion': [
       'error',
       {
@@ -45,6 +47,14 @@ module.exports = {
         ignorePropertyModificationsFor: ['acc', 'accumulator', 'el', 'element', 'state'],
       },
     ],
+    'no-restricted-exports': [
+      'error',
+      {
+        restrictedNamedExports: [
+          'then', // avoid confusion when used with dynamic `import()` promise
+        ],
+      },
+    ],
     'no-sequences': [
       'error',
       {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gitlab/eslint-plugin",
-  "version": "12.1.0",
+  "version": "12.3.0",
   "description": "GitLab package for our custom eslint rules",
   "main": "lib/index.js",
   "scripts": {
@@ -33,10 +33,10 @@
     "@babel/core": "^7.17.0",
     "@babel/eslint-parser": "^7.17.0",
     "@babel/eslint-plugin": "^7.17.7",
-    "eslint-config-airbnb-base": "^14.2.1",
+    "eslint-config-airbnb-base": "^15.0.0",
     "eslint-config-prettier": "^6.10.0",
     "eslint-plugin-babel": "^5.3.0",
-    "eslint-plugin-import": "^2.22.1",
+    "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-jest": "^23.8.2",
     "eslint-plugin-promise": "^4.2.1",
     "eslint-plugin-unicorn": "^40.1.0",
@@ -44,7 +44,7 @@
     "lodash": "^4.17.21"
   },
   "peerDependencies": {
-    "eslint": "^7.27.0"
+    "eslint": "^7.27.0 || ^8.0.0"
   },
   "devDependencies": {
     "eslint": "^7.27.0",

package/scripts/integration_test/bootstrap.sh

@@ -36,3 +36,9 @@ fi
 
 echo "Installing $CI_REPOSITORY_URL#$SHA"
 yarn add --dev "${CI_REPOSITORY_URL}#${SHA}"
+
+echo "Deduplicate dependencies"
+# Yarn doesn't seem to be smart enough when using URL installs and might
+# end up with a weird state, retaining outdated dependencies
+npx yarn-deduplicate --strategy fewer yarn.lock
+yarn install