code: Use object shorthand for properties

.eslintrc.json

@@ -17,6 +17,7 @@
 		"eslint-plugin/require-meta-docs-url": "off",
 		"eslint-plugin/test-case-property-ordering": [ "error",
 			[ "code", "options", "parserOptions", "settings", "errors", "output" ]
-		]
+		],
+		"object-shorthand": [ "error", "properties" ]
 	}
 }

src/rules/no-ajax-events.js

@@ -43,7 +43,7 @@ module.exports = {
 			}
 			if ( usedMethod && utils.isjQuery( context, node ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: 'Prefer local event to {{method}}',
 					data: { method: usedMethod }
 				} );

src/rules/no-animate-toggle.js

@@ -45,7 +45,7 @@ module.exports = {
 
 				if ( utils.isjQuery( context, node ) ) {
 					context.report( {
-						node: node,
+						node,
 						message: 'Prefer CSS transitions to .{{method}}',
 						data: { method: node.callee.property.name }
 					} );

src/rules/no-animate.js

@@ -48,7 +48,7 @@ module.exports = {
 
 			if ( utils.isjQuery( context, node ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: allowScroll ?
 						'Prefer CSS transitions to .animate' :
 						'Prefer CSS transitions or CSS scroll-behaviour to .animate'

src/rules/no-append-html.js

@@ -42,7 +42,7 @@ module.exports = {
 
 			if ( utils.isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: 'Avoid injection of possibly unescaped HTML. Create DOM elements instead, or use .text.'
 				} );
 			}

src/rules/no-class-state.js

@@ -34,7 +34,7 @@ module.exports = {
 
 			if ( utils.isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: 'Where possible, maintain application state in JS to avoid slower DOM queries'
 				} );
 			}

src/rules/no-constructor-attributes.js

@@ -35,7 +35,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: 'Prefer .attr to constructor attributes'
 			} );
 		}

src/rules/no-deferred.js

@@ -22,7 +22,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: 'Prefer Promise to $.Deferred'
 			} );
 		}

src/rules/no-extend.js

@@ -41,7 +41,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: 'Prefer Object.assign or the spread operator to $.extend',
 				fix: function ( fixer ) {
 					if ( !isDeep ) {

src/rules/no-fx-interval.js

@@ -23,7 +23,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: '$.fx.interval is not allowed'
 			} );
 		}

src/rules/no-global-selector.js

@@ -78,7 +78,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: 'Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible.'
 			} );
 		}

src/rules/no-jquery-constructor.js

@@ -21,7 +21,7 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: 'The jQuery constructor is not allowed'
 			} );
 		}

src/rules/no-load-shorthand.js

@@ -27,7 +27,7 @@ module.exports = {
 
 			if ( utils.isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: 'Prefer .on or .trigger to .load',
 					fix: utils.eventShorthandFixer.bind( this, node, context )
 				} );

src/rules/no-on-ready.js

@@ -27,7 +27,7 @@ module.exports = {
 
 			if ( utils.isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: '.on("ready") is not allowed',
 					fix: ( fixer ) => ( node.arguments.length > 1 ) ?
 						[

src/rules/no-other-methods.js

@@ -109,9 +109,9 @@ module.exports = {
 			}
 			if ( utils.isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: '.{{name}} is not allowed',
-					data: { name: name }
+					data: { name }
 				} );
 			}
 		}

src/rules/no-other-utils.js

@@ -77,9 +77,9 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: '$.{{name}} is not allowed',
-				data: { name: name }
+				data: { name }
 			} );
 		}
 	} )

src/rules/no-parse-html-literal.js

@@ -128,8 +128,8 @@ module.exports = {
 			}
 
 			context.report( {
-				node: node,
-				message: message,
+				node,
+				message,
 				fix: ( fixer ) => expectedTag ? fixer.replaceText( arg, '"' + expectedTag + '"' ) : null
 			} );
 		}

src/rules/no-ready.js

@@ -34,7 +34,7 @@ module.exports = {
 		'CallExpression:exit': ( node ) => {
 			if ( isDirect( context, node ) || isChained( context, node ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: '.ready is not allowed'
 				} );
 			}

src/rules/no-sizzle.js

@@ -83,12 +83,12 @@ module.exports = {
 
 				if ( !allowPositional && forbiddenPositional.test( value ) ) {
 					context.report( {
-						node: node,
+						node,
 						message: 'Positional selector extensions are not allowed'
 					} );
 				} else if ( !allowOther && forbiddenOther.test( value ) ) {
 					context.report( {
-						node: node,
+						node,
 						message: 'Selector extensions are not allowed'
 					} );
 				}

src/rules/variable-pattern.js

@@ -22,7 +22,7 @@ module.exports = {
 				right && utils.isjQuery( context, right )
 			) {
 				context.report( {
-					node: node,
+					node,
 					message: 'jQuery collection names must match the variablePattern'
 				} );
 			}

src/utils.js

@@ -173,7 +173,7 @@
 //
 // Returns true if the function call node is attached to a jQuery element set.
 function isjQuery( context, node ) {
 	const variablePattern = new RegExp(
 		( context.settings && context.settings[ 'no-jquery' ] && context.settings[ 'no-jquery' ].variablePattern ) ||
 		'^\\$.'
 	);
@@ -203,14 +203,14 @@
 		meta: {
 			type: 'suggestion',
 			docs: {
-				description: description,
+				description,
 				deprecated: !!deprecated,
 				replacedBy: Array.isArray( deprecated ) ? deprecated : undefined
 			},
-			fixable: fixable,
+			fixable,
 			schema: schema || []
 		},
-		create: create
+		create
 	};
 }
 
@@ -359,7 +359,7 @@
 
 			if ( isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: messageToPlainString( message, node, name, options ),
 					fix: options.fix && options.fix.bind( this, node, context )
 				} );
@@ -396,7 +396,7 @@
 			}
 			if ( isjQuery( context, node.object ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: messageToPlainString( message, node, name, options ),
 					fix: options.fix && options.fix.bind( this, node, context )
 				} );
@@ -439,7 +439,7 @@
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: messageToPlainString( message, node, name, options ),
 				fix: options.fix && options.fix.bind( this, node, context )
 			} );
@@ -475,7 +475,7 @@
 			}
 
 			context.report( {
-				node: node,
+				node,
 				message: messageToPlainString( message, node, name, options ),
 				fix: options.fix && options.fix.bind( this, node, context )
 			} );
@@ -517,7 +517,7 @@
 			}
 			if ( isjQuery( context, node.callee ) ) {
 				context.report( {
-					node: node,
+					node,
 					message: messageToPlainString( message, node, name, options ),
 					fix: options.fix && options.fix.bind( this, node, context )
 				} );
@@ -542,15 +542,15 @@
 }
 
 module.exports = {
-	isjQuery: isjQuery,
-	isjQueryConstructor: isjQueryConstructor,
-	isFunction: isFunction,
-	createCollectionMethodRule: createCollectionMethodRule,
-	createCollectionPropertyRule: createCollectionPropertyRule,
-	createUtilMethodRule: createUtilMethodRule,
-	createUtilPropertyRule: createUtilPropertyRule,
-	createCollectionOrUtilMethodRule: createCollectionOrUtilMethodRule,
-	eventShorthandFixer: eventShorthandFixer,
-	jQueryCollectionLink: jQueryCollectionLink,
-	jQueryGlobalLink: jQueryGlobalLink
+	isjQuery,
+	isjQueryConstructor,
+	isFunction,
+	createCollectionMethodRule,
+	createCollectionPropertyRule,
+	createUtilMethodRule,
+	createUtilPropertyRule,
+	createCollectionOrUtilMethodRule,
+	eventShorthandFixer,
+	jQueryCollectionLink,
+	jQueryGlobalLink
 };

tests/rules/no-event-shorthand.js

@@ -121,5 +121,5 @@ ruleTester.run( 'no-event-shorthand', rule, {
 			options: [ { allowAjaxEvents: true } ]
 		} ) )
 	),
-	invalid: invalid
+	invalid
 } );