improve(COURIER-1006): Husky updates

Author: aaronware

.husky/commit-msg

@@ -0,0 +1,4 @@
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+npx commitlint --edit $1 

.husky/pre-commit

@@ -0,0 +1,15 @@
+#!/bin/sh
+. "$(dirname "$0")/_/husky.sh"
+
+# Get staged PHP files
+STAGED_PHP_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep '\.php$')
+
+if [ -z "$STAGED_PHP_FILES" ]; then
+  exit 0
+fi
+
+# Run phpcbf on staged files
+./vendor/bin/phpcbf --standard=phpcs.xml.dist $STAGED_PHP_FILES
+
+# Add the fixed files to the commit
+echo "$STAGED_PHP_FILES" | xargs git add 

.php-cs-fixer.dist.php

@@ -26,8 +26,8 @@
 		'native_function_casing'         => true,
 		'native_function_invocation'     => true,
 		'native_type_declaration_casing' => true,
-		'array_syntax'                   => ['syntax' => 'short'],
-		'list_syntax'                    => ['syntax' => 'short'],
+		'array_syntax'                   => [ 'syntax' => 'short' ],
+		'list_syntax'                    => [ 'syntax' => 'short' ],
 	]
 );
 $config->setFinder( $finder );

commitlint.config.js

@@ -6,20 +6,38 @@
  *
  * Format Example
  *
- * issue(MY-123): Add new feature
+ * issue(COURIER-123): Add new feature
  *
- * or if you have no issue number you can utilize NO-JIRA or NO-TASK
+ * or if you have no issue number you can utilize NO-TASK
  */
 module.exports = {
-	extends: ['@commitlint/config-conventional'],
+	extends: ["@commitlint/config-conventional"],
 	rules: {
-		'type-enum': [2, 'always', ['improve', 'build','chore','ci','docs','feat','fix','perf','refactor','revert','style','test']],
-		'subject-case': [1, 'always', ['sentence-case']],
+		"type-enum": [
+			2,
+			"always",
+			[
+				"improve",
+				"build",
+				"chore",
+				"ci",
+				"docs",
+				"feat",
+				"fix",
+				"perf",
+				"refactor",
+				"revert",
+				"style",
+				"test",
+			],
+		],
+		"subject-case": [1, "always", ["sentence-case"]],
+	},
+	parserPreset: {
+		parserOpts: {
+			headerPattern:
+				/^(improve|build|ci|feat|fix|docs|style|revert|perf|refactor|test|chore)\(((?:COURIER-\d+)|(?:NO-JIRA|NO-TASK)|(?:\#\d+))\):\s?([\w\d\s,\-]*)/,
+			headerCorrespondence: ["type", "scope", "subject"],
+		},
 	},
-	"parserPreset": {
-		"parserOpts": {
-			"headerPattern": /^(improve|build|ci|feat|fix|docs|style|revert|perf|refactor|test|chore)\(((?:[A-Z]+-\d+)|(?:NO-JIRA|NO-TASK)|(?:\#\d+))\):\s?([\w\d\s,\-]*)/,
-			"headerCorrespondence": ["type", "scope", "subject"]
-		}
-	}
 };

courier-notices.php

@@ -69,7 +69,7 @@
 	require_once COURIER_NOTICES_PATH . 'vendor/autoload.php';
 }
 
-// Load Strauss prefixed dependencies
+// Load Strauss prefixed dependencies.
 if ( file_exists( COURIER_NOTICES_PATH . 'vendor-prefixed/autoload.php' ) ) {
 	require_once COURIER_NOTICES_PATH . 'vendor-prefixed/autoload.php';
 }
@@ -90,8 +90,10 @@
 function courier_notices_init() {
 
 	// Deprecated hook.
-	_doing_it_wrong( 'before_courier_notices_init', 'Use courier_notices_before_init instead.', '1.7.0' );
-	do_action( 'before_courier_notices_init' ); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound
+	if ( has_action( 'before_courier_notices_init' ) ) {
+		_doing_it_wrong( 'before_courier_notices_init', 'Use courier_notices_before_init instead.', '1.7.0' );
+		do_action( 'before_courier_notices_init' ); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound
+	}
 
 	do_action( 'courier_notices_before_init' );
 
@@ -118,8 +120,10 @@ function () {
 	}
 
 	// Deprecated hook.
-	_doing_it_wrong( 'after_courier_notices_init', 'Use courier_notices_after_init instead.', '1.7.0' );
-	do_action( 'after_courier_notices_init' ); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound
+	if ( has_action( 'after_courier_notices_init' ) ) {
+		_doing_it_wrong( 'after_courier_notices_init', 'Use courier_notices_after_init instead.', '1.7.0' );
+		do_action( 'after_courier_notices_init' ); // phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound
+	}
 
 	do_action( 'courier_notices_after_init' );
 }

package-lock.json

@@ -45,5 +45,6 @@
 	"browserslist": [
 		"defaults"
 	],
-	"private": true
+	"private": true,
+	"prepare": "husky || true"
 }