Merge pull request #169 from humanmade/153-enforce-semicolons

ESLint Enforce semicolons

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## Unreleased (0.8.0)
 
 ### Added:
+ - Enforce semicolons in JS #169
  - Added `WordPress.Security.EscapeOutput` PHPCS rule #166
 
 ### Updated

packages/eslint-config-humanmade/fixtures/fail/semicolon.js

@@ -0,0 +1,14 @@
+/* eslint-disable no-unused-vars */
+
+// Missing semicolon.
+const testMissingSemicolon = 'foo'
+
+// Incorrect space before semicolon.
+const testSemiSpacing = 'bar' ;
+
+// Double semicolon.
+const testDoubleSemiColon = 5;;
+
+const testFunc = function () {
+	// ...
+}

packages/eslint-config-humanmade/fixtures/pass/semicolon.js

@@ -0,0 +1,11 @@
+/* eslint-disable no-unused-vars */
+
+// Missing semicolon.
+const testMissingSemicolon = 'foo';
+
+// Double semicolon.
+const testDoubleSemiColon = 5;
+
+const testFunc = function () {
+	// ...
+};

packages/eslint-config-humanmade/index.js

@@ -74,6 +74,7 @@ module.exports = {
 		'object-curly-spacing': [ 'error', 'always' ],
 		'object-property-newline': [ 'error' ],
 		'quotes': [ 'error', 'single' ],
+		'semi': [ 'error', 'always' ],
 		'semi-spacing': [ 'error', {
 			'before': false,
 			'after': true,