Skip to content

Commit

Permalink
Remove unneeded parserOptions from eslintrc files
Browse files Browse the repository at this point in the history
We switched to using `@typescript-eslint/parser` in #23522. This parser
does not use the options `babelOptions`, `sourceType`, or
`requireConfigFile`, so let's remove them.
  • Loading branch information
anomiex committed Aug 28, 2024
1 parent 1b98f4d commit 49deb33
Show file tree
Hide file tree
Showing 63 changed files with 150 additions and 156 deletions.
4 changes: 0 additions & 4 deletions projects/js-packages/boost-score-api/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@ module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/base' ) ],
ignorePatterns: loadIgnorePatterns( __dirname ),
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.cjs' ),
},
sourceType: 'module',
tsconfigRootDir: __dirname,
project: [ './tsconfig.json' ],
},
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/components/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/connection/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/idc/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


1 change: 0 additions & 1 deletion projects/js-packages/image-guide/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ module.exports = {
],
ignorePatterns: loadIgnorePatterns( __dirname ),
parserOptions: {
sourceType: 'module',
tsconfigRootDir: __dirname,
project: [ './tsconfig.json' ],
},
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/licensing/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/partner-coupon/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/publicize-components/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/shared-extension-utils/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/js-packages/storybook/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
presets: [ require.resolve( '@babel/preset-react' ) ],
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/backup/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce the use of the jetpack-backup textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/blaze/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,4 @@
module.exports = {
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
'jsdoc/check-tag-names': [ 1, { definedTags: [ 'jest-environment' ] } ],
// Enforce use of the correct textdomain.
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


6 changes: 0 additions & 6 deletions projects/packages/jetpack-mu-wpcom/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
requireConfigFile: false,
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
'jsdoc/require-returns': 0,
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/my-jetpack/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/publicize/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/search/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
'jsdoc/check-tag-names': [ 1, { definedTags: [ 'jsx', 'jest-environment' ] } ],
'react/jsx-no-bind': 0,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/wordads/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
'jsdoc/check-tag-names': [ 1, { definedTags: [ 'jsx', 'jest-environment' ] } ],
'react/jsx-no-bind': 0,
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/packages/yoast-promo/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,4 @@
module.exports = {
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
'jsdoc/check-tag-names': [ 1, { definedTags: [ 'jest-environment' ] } ],
// Enforce use of the correct textdomain.
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/plugins/automattic-for-agencies-client/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce the use of the automattic-for-agencies-client textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


4 changes: 0 additions & 4 deletions projects/plugins/boost/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,6 @@ module.exports = {
],
ignorePatterns: [ '**/stories/*.stories.tsx', ...loadIgnorePatterns( __dirname ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
sourceType: 'module',
tsconfigRootDir: __dirname,
project: [ './tsconfig.json', './tsconfig.eslint.json' ],
},
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/plugins/classic-theme-helper-plugin/.eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,5 @@
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce the use of the classic-theme-helper-plugin textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


4 changes: 0 additions & 4 deletions projects/plugins/inspect/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@ module.exports = {
],
ignorePatterns: loadIgnorePatterns( __dirname ),
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
sourceType: 'module',
tsconfigRootDir: __dirname,
project: [ './tsconfig.json' ],
},
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: removed
Comment: Remove unneeded `parserOptions` from eslintrc.


5 changes: 0 additions & 5 deletions projects/plugins/jetpack/.eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,6 @@
*/
module.exports = {
extends: [ require.resolve( 'jetpack-js-tools/eslintrc/react' ) ],
parserOptions: {
babelOptions: {
configFile: require.resolve( './babel.config.js' ),
},
},
rules: {
// Enforce use of the correct textdomain.
'@wordpress/i18n-text-domain': [
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Significance: patch
Type: other
Comment: Remove unneeded `parserOptions` from eslintrc.


Loading

0 comments on commit 49deb33

Please sign in to comment.