code: Use object shorthand for properties (#315)

* code: Use object shorthand for properties

* .eslintrc.json: Drop upstream duplicate config

---------

Co-authored-by: James D. Forrester <[email protected]>

Author: edg2s

.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'
 			} );
 		}