chore: Remove NOSCAN directive in favor of phpcs:ignore.

wpengine · Jan 29, 2024 · 4cdeee9 · 4cdeee9
1 parent 7440723
commit 4cdeee9
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/includes/WPGraphQLContentBlocks.php b/includes/WPGraphQLContentBlocks.php
@@ -150,10 +150,9 @@ static function () {
 		require_once WPGRAPHQL_CONTENT_BLOCKS_PLUGIN_DIR . 'includes/updates/update-functions.php';
 		require_once WPGRAPHQL_CONTENT_BLOCKS_PLUGIN_DIR . 'includes/updates/update-callbacks.php';
 
-		// Tell SonarQube to ignore the following line. The library bootstraps itself, hence no need to instantiate to a variable.
-		// BEGIN-NOSCAN
-		new \EnforceSemVer\EnforceSemVer( WPGRAPHQL_CONTENT_BLOCKS_PATH );
-		// END-NOSCAN
+		// phpcs:ignore SlevomatCodingStandard.Variables.UnusedVariable.UnusedVariable -- Library bootstraps itself, hence variable is unused.
+		$semver = new \EnforceSemVer\EnforceSemVer( WPGRAPHQL_CONTENT_BLOCKS_PATH );
+
 		return true;
 	}