@gitlab/eslint-plugin 12.0.1 → 12.2.1

package/.editorconfig

@@ -0,0 +1,18 @@
+# top-most EditorConfig file
+root = true
+
+# Unix-style newlines with a newline ending every file
+[*]
+end_of_line = lf
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+[*.{js,json,vue,scss,rb,haml,yml}]
+indent_size = 2
+
+[*.{js,json,vue,scss,rb,haml,yml,md}]
+indent_style = space
+charset = utf-8
+
+[*.{md,markdown,js.snap}]
+trim_trailing_whitespace = false

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+## [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)
+
+
+### Features
+
+* update eslint-plugin-vue to 8.5.0 ([37ac645](https://gitlab.com/gitlab-org/frontend/eslint-plugin/commit/37ac645f4421ce06b083625c53356e9f6c7c5679))
+
 ## [12.0.1](https://gitlab.com/gitlab-org/frontend/eslint-plugin/compare/v12.0.0...v12.0.1) (2022-03-28)
 
 

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,6 +26,7 @@ module.exports = {
         ignoreDestructuring: true,
       },
     ],
+    'default-param-last': 'off',
     'unicorn/filename-case': ['error', { case: 'snakeCase' }],
     'import/prefer-default-export': 'off',
     'import/no-default-export': 'error',
@@ -45,6 +46,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.0.1",
+  "version": "12.2.1",
   "description": "GitLab package for our custom eslint rules",
   "main": "lib/index.js",
   "scripts": {
@@ -33,26 +33,26 @@
     "@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.25.2",
     "eslint-plugin-jest": "^23.8.2",
     "eslint-plugin-promise": "^4.2.1",
     "eslint-plugin-unicorn": "^40.1.0",
-    "eslint-plugin-vue": "^7.5.0",
-    "lodash": "^4.17.21",
-    "vue-eslint-parser": "^7.0.0"
+    "eslint-plugin-vue": "^8.5.0",
+    "lodash": "^4.17.21"
   },
   "peerDependencies": {
-    "eslint": "^7.27.0"
+    "eslint": "^7.27.0 || ^8.0.0"
   },
   "devDependencies": {
     "eslint": "^7.27.0",
     "glob": "^7.2.0",
     "jest": "^27.5.1",
     "prettier": "^2.6.1",
-    "pretty-quick": "^3.1.3"
+    "pretty-quick": "^3.1.3",
+    "vue-eslint-parser": "^8.0.1"
   },
   "release": {
     "branches": [

package/scripts/integration_test/bootstrap.sh

@@ -0,0 +1,44 @@
+#!/usr/bin/env bash
+# Use the unofficial bash strict mode: http://redsymbol.net/articles/unofficial-bash-strict-mode/
+set -euo pipefail; export FS=$'\n\t'
+
+SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
+SRC_DIR="$SCRIPT_DIR/../../src"
+
+if [ "$CI" != "true" ]; then
+    echo "This script is meant to run in CI"
+    exit 1
+fi
+
+if [ -d "$SRC_DIR" ]; then
+    cd "$SRC_DIR" || exit 1
+    REMOTE=$(git remote)
+    DEFAULT_BRANCH=$(git symbolic-ref "refs/remotes/${REMOTE}/HEAD" | rev | cut -d '/' -f1 | rev)
+    echo "src/ already exists, pulling default branch (${DEFAULT_BRANCH})"
+    git pull "$REMOTE" "$DEFAULT_BRANCH"
+else
+    echo "Cloning ${TARGET_REPO} into src/"
+    git clone --depth 1 "${TARGET_REPO}" "$SRC_DIR";
+    cd "$SRC_DIR" || exit 1
+fi
+
+
+echo "Installing dependencies"
+yarn install --frozen-lockfile
+
+if [ -n "${CI_MERGE_REQUEST_SOURCE_BRANCH_SHA:-}" ]; then
+    echo "We seem to be on a Merge Request..."
+    SHA="$CI_MERGE_REQUEST_SOURCE_BRANCH_SHA"
+else
+    echo "We seem to be on a branch..."
+    SHA="$CI_COMMIT_SHA"
+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