Merge branch 'master' into nav

.eslintignore

@@ -2,9 +2,13 @@
 **/*.build.js
 **/node_modules/**
 **/vendor/**
+dist
 build
-coverage
-cypress
-node_modules
-vendor
+gulp
 bin
+includes/assets/js/
+src/packages/
+src/assets/js/scroll-magic.js
+src/assets/js/progressBar.js
+src/assets/js/countUp.js
+src/assets/js/kt-table-of-contents.js

.eslintrc.js

@@ -12,21 +12,24 @@ const eslintConfig = {
 			},
 		},
 	},
+	globals: {
+		kt_blocks_params: 'readonly',
+		kadence_blocks_params: 'writeable',
+		kt_deregister_params: 'readonly',
+		kadence_blocks_user_params: 'readonly',
+	},
 	rules: {
-		'import/no-extraneous-dependencies': 'off',
-		'import/no-unresolved': 'off',
-		'import/named': 'off',
 		// '@wordpress/i18n-text-domain': [    --- From here down we should start enabling rules and fixing errors
 		// 	'error',
 		// 	{
 		// 		allowedTextDomain: ['kadence-blocks'],
 		// 	},
 		// ],
+		'import/no-extraneous-dependencies': 'off',
+		'import/no-unresolved': 'off',
+		'import/named': 'off',
 		'no-unused-vars': 'off',
 		'react-hooks/exhaustive-deps': 'off',
-		'dot-notation': 'off',
-		'object-shorthand': 'off',
-		'prefer-const': 'off',
 		'no-shadow': 'off',
 		'@wordpress/i18n-ellipsis': 'off',
 		'jsdoc/require-param': 'off',
@@ -37,30 +40,24 @@ const eslintConfig = {
 		'react-hooks/rules-of-hooks': 'off',
 		'no-console': 'off',
 		'react/jsx-no-target-blank': 'off',
+		'no-var': 'off',
 		'array-callback-return': 'off',
 		'jsdoc/check-types': 'off',
 		'jsdoc/check-tag-names': 'off',
 		'jsdoc/newline-after-description': 'off',
-		'no-duplicate-imports': 'off',
 		'jsx-a11y/label-has-associated-control': 'off',
 		eqeqeq: 'off',
 		'no-unused-expressions': 'off',
 		'jsx-a11y/no-autofocus': 'off',
-		'no-var': 'off',
-		'no-unreachable': 'off',
 		'@wordpress/i18n-translator-comments': 'off',
 		'jsdoc/check-line-alignment': 'off',
 		'no-useless-computed-key': 'off',
 		'@wordpress/no-unused-vars-before-return': 'off',
 		'react/no-unknown-property': 'off',
-		'no-dupe-keys': 'off',
 		'react/jsx-key': 'off',
 		'jsdoc/check-param-names': 'off',
 		'jsdoc/no-undefined-types': 'off',
-		'react/jsx-no-undef': 'off',
 		'jsx-a11y/click-events-have-key-events': 'off',
-		'no-lonely-if': 'off',
-		'no-else-return': 'off',
 		'jsdoc/valid-types': 'off',
 		'jsx-a11y/interactive-supports-focus': 'off',
 		'jsdoc/require-returns-description': 'off',
@@ -78,12 +75,11 @@ const eslintConfig = {
 		'jsx-a11y/iframe-has-title': 'off',
 		'jsdoc/empty-tags': 'off',
 		'no-bitwise': 'off',
-		'no-cond-assign': 'off',
 		'jsdoc/require-returns-check': 'off',
 		'@wordpress/no-global-active-element': 'off',
 		'jsx-a11y/anchor-has-content': 'off',
 	},
-	ignorePatterns: ['node_modules/', 'dist/', 'build/', 'vendor/', 'gulp/', 'src/packages/'],
+	ignorePatterns: [],
 };
 
 module.exports = eslintConfig;

.git-blame-ignore-revs

@@ -10,3 +10,4 @@
 
 # Apply Prettier formatting
 5484ce8508d62006269fc6a50d89790104ec6371
+ed41efaa02bb759f1be83a19ec87e76e8acc2b88

.github/workflows/eslint.yml

@@ -0,0 +1,16 @@
+name: ESLint
+run-name: '[ESLint] ${{ github.event.head_commit.message }}'
+
+on: push
+jobs:
+    lint:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v4
+            - uses: actions/setup-node@v4
+              with:
+                  node-version: 18
+            - run: npm ci --ignore-scripts --legacy-peer-deps
+            - run: npm rebuild && npm run prepare --if-present
+            - name: Run ESLint
+              run: npm run lint-js

.gitignore

@@ -5,6 +5,7 @@ dist/
 zip/
 bin/strauss.phar
 .idea
+.vscode
 /vendor/
 /vendor-prefixed/
 bin/pup.phar

.prettierignore

@@ -1,2 +1,4 @@
 **/*.min.js
 /src/packages/**
+**/node_modules/**
+**/vendor/**

gulp/tasks/js.js

@@ -39,6 +39,7 @@ function copyVendorFiles() {
 				};
 			})
 		)
+		.pipe(minify(config.minify))
 		.pipe(dest(config.dirs.dist + '/js'));
 }