Remove no longer necessary globals from the Eslint config

.eslintrc.js

@@ -20,13 +20,7 @@ module.exports = {
 	root: true,
 	extends: [
 		'@wordpress/eslint-config',
-		'plugin:jest/recommended'
-	],
-	env: {
-		'jest/globals': true,
-	},
-	plugins: [
-		'jest',
+		'plugin:jest/recommended',
 	],
 	rules: {
 		'no-restricted-syntax': [
@@ -189,9 +183,13 @@ module.exports = {
 	overrides: [
 		{
 			files: [ 'test/e2e/**/*.js' ],
+			env: {
+				browser: true,
+			},
 			globals: {
-				page: true,
 				browser: true,
+				page: true,
+				wp: true,
 			},
 		},
 	],

packages/block-library/src/classic/edit.js

@@ -5,6 +5,8 @@ import { Component } from '@wordpress/element';
 import { __, _x } from '@wordpress/i18n';
 import { BACKSPACE, DELETE, F10 } from '@wordpress/keycodes';
 
+const { wp } = window;
+
 function isTmceEmpty( editor ) {
 	// When tinyMce is empty the content seems to be:
 	// <p><br data-mce-bogus="1"></p>

packages/edit-post/src/hooks/components/media-upload/index.js

@@ -1,14 +1,16 @@
 /**
  * External Dependencies
  */
-import { castArray, pick } from 'lodash';
+import { castArray, defaults, pick } from 'lodash';
 
 /**
  * WordPress dependencies
  */
 import { Component } from '@wordpress/element';
 import { __ } from '@wordpress/i18n';
 
+const { wp } = window;
+
 // Getter for the sake of unit tests.
 const getGalleryDetailsMediaFrame = () => {
 	/**
@@ -36,7 +38,7 @@ const getGalleryDetailsMediaFrame = () => {
 					multiple: 'add',
 					editable: false,
 
-					library: wp.media.query( _.defaults( {
+					library: wp.media.query( defaults( {
 						type: 'image',
 					}, this.options.library ) ),
 				} ),

packages/eslint-config/configs/es5.js

@@ -0,0 +1,12 @@
+/**
+ * The original version of this file is based on WordPress ESLint rules and shared configs:
+ * https://github.com/WordPress-Coding-Standards/eslint-plugin-wordpress.
+ */
+
+module.exports = {
+	env: {
+		es6: true,
+	},
+
+	rules: require( './rules/esnext' ),
+};

packages/eslint-config/configs/esnext.js

@@ -4,9 +4,5 @@
  */
 
 module.exports = {
-	env: {
-		es6: true,
-	},
-
-	rules: require( './rules/esnext' ),
+	rules: require( './rules/es5' ),
 };

packages/eslint-config/configs/rules/es5.js

@@ -0,0 +1,84 @@
+module.exports = {
+	// Possible Errors
+	// Disallow assignment in conditional expressions
+	'no-cond-assign': [ 'error', 'except-parens' ],
+	// Disallow irregular whitespace outside of strings and comments
+	'no-irregular-whitespace': 'error',
+	// Best Practices
+	// Specify curly brace conventions for all control statements
+	curly: [ 'error', 'all' ],
+	// Encourages use of dot notation whenever possible
+	'dot-notation': [ 'error', {
+		allowKeywords: true,
+		allowPattern: '^[a-z]+(_[a-z]+)+$',
+	} ],
+	// Disallow use of multiline strings
+	'no-multi-str': 'error',
+	// Disallow use of the with statement
+	'no-with': 'error',
+	// Requires to declare all vars on top of their containing scope
+	'vars-on-top': 'error',
+	// Require immediate function invocation to be wrapped in parentheses
+	'wrap-iife': 'error',
+	// Require or disallow Yoda conditions
+	yoda: [ 'error', 'always' ],
+	// Strict Mode
+	// Variables
+	// Stylistic Issues
+	// Enforce spacing inside array brackets
+	'array-bracket-spacing': [ 'error', 'always' ],
+	// Enforce one true brace style
+	'brace-style': 'error',
+	// Require camel case names
+	camelcase: [ 'error', {
+		properties: 'always',
+	} ],
+	// Disallow or enforce trailing commas
+	'comma-dangle': [ 'error', 'never' ],
+	// Enforce spacing before and after comma
+	'comma-spacing': 'error',
+	// Enforce one true comma style
+	'comma-style': [ 'error', 'last' ],
+	// Enforce newline at the end of file, with no multiple empty lines
+	'eol-last': 'error',
+	// Enforces spacing between keys and values in object literal properties
+	'key-spacing': [ 'error', {
+		beforeColon: false,
+		afterColon: true,
+	} ],
+	// Enforce spacing before and after keywords
+	'keyword-spacing': 'error',
+	// Disallow mixed "LF" and "CRLF" as linebreaks
+	'linebreak-style': [ 'error', 'unix' ],
+	// Enforces empty lines around comments
+	'lines-around-comment': [ 'error', {
+		beforeLineComment: true,
+	} ],
+	// Disallow mixed spaces and tabs for indentation
+	'no-mixed-spaces-and-tabs': 'error',
+	// Disallow multiple empty lines
+	'no-multiple-empty-lines': 'error',
+	// Disallow trailing whitespace at the end of lines
+	'no-trailing-spaces': 'error',
+	// Require or disallow an newline around variable declarations
+	'one-var-declaration-per-line': [ 'error', 'initializations' ],
+	// Enforce operators to be placed before or after line breaks
+	'operator-linebreak': [ 'error', 'after' ],
+	// Specify whether backticks, double or single quotes should be used
+	quotes: [ 'error', 'single' ],
+	// Require or disallow use of semicolons instead of ASI
+	semi: [ 'error', 'always' ],
+	// Require or disallow space before blocks
+	'space-before-blocks': [ 'error', 'always' ],
+	// Require or disallow space before function opening parenthesis
+	'space-before-function-paren': [ 'error', 'never' ],
+	// Require or disallow space before blocks
+	'space-in-parens': [ 'error', 'always', { exceptions: [ '{}', '[]' ] } ],
+	// Require spaces around operators
+	'space-infix-ops': 'error',
+	// Require or disallow spaces before/after unary operators (words on by default, nonwords)
+	'space-unary-ops': [ 'error', {
+		overrides: { '!': true },
+	} ],
+	// Legacy
+};

packages/eslint-config/index.js

@@ -1,13 +1,12 @@
 module.exports = {
 	parser: 'babel-eslint',
 	extends: [
-		'./configs/wordpress.js',
+		'./configs/es5.js',
 		'./configs/esnext.js',
 		'plugin:react/recommended',
 		'plugin:jsx-a11y/recommended',
 	],
 	env: {
-		browser: false,
 		node: true,
 	},
 	parserOptions: {