Remove command for minimal plugin standard
Because we used outdated WP coding standard we had reported errors about new WPDB placeholder %i. [MAILPOET-6150]
This commit is contained in:
@@ -621,9 +621,6 @@ class RoboFile extends \Robo\Tasks {
|
|||||||
$collection->addCode(function() {
|
$collection->addCode(function() {
|
||||||
return $this->qaCodeSniffer([]);
|
return $this->qaCodeSniffer([]);
|
||||||
});
|
});
|
||||||
$collection->addCode(function() {
|
|
||||||
return $this->qaMinimalPluginStandard([]);
|
|
||||||
});
|
|
||||||
return $collection->run();
|
return $collection->run();
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -719,6 +716,7 @@ class RoboFile extends \Robo\Tasks {
|
|||||||
'tasks/code_sniffer/vendor',
|
'tasks/code_sniffer/vendor',
|
||||||
'tasks/phpstan/vendor',
|
'tasks/phpstan/vendor',
|
||||||
'tasks/makepot',
|
'tasks/makepot',
|
||||||
|
'tasks/minimal-plugin-standard/vendor',
|
||||||
'tools/vendor',
|
'tools/vendor',
|
||||||
'tools/wpscan-semgrep-rules',
|
'tools/wpscan-semgrep-rules',
|
||||||
'temp',
|
'temp',
|
||||||
@@ -743,58 +741,6 @@ class RoboFile extends \Robo\Tasks {
|
|||||||
->run();
|
->run();
|
||||||
}
|
}
|
||||||
|
|
||||||
public function qaMinimalPluginStandard(array $filesToCheck, $opts = ['severity' => 'all']) {
|
|
||||||
$severityFlag = $opts['severity'] === 'all' ? '-w' : '-n';
|
|
||||||
|
|
||||||
$task = implode(' ', [
|
|
||||||
'php -d memory_limit=-1',
|
|
||||||
'./tasks/code_sniffer/vendor/bin/phpcs',
|
|
||||||
'--parallel=' . $this->getParallelism(),
|
|
||||||
'--extensions=php',
|
|
||||||
$severityFlag,
|
|
||||||
'--standard=tasks/code_sniffer/vendor/wporg/plugin-directory/MinimalPluginStandard',
|
|
||||||
'-s',
|
|
||||||
]);
|
|
||||||
|
|
||||||
$ignorePaths = [
|
|
||||||
'.mp_svn',
|
|
||||||
'assets',
|
|
||||||
'doc',
|
|
||||||
'generated',
|
|
||||||
'lib/Config/PopulatorData/Templates',
|
|
||||||
'lib-3rd-party',
|
|
||||||
'node_modules',
|
|
||||||
'plugin_repository',
|
|
||||||
'prefixer/build',
|
|
||||||
'prefixer/vendor',
|
|
||||||
'tasks/code_sniffer/vendor',
|
|
||||||
'tasks/phpstan/vendor',
|
|
||||||
'tasks/makepot',
|
|
||||||
'tools/vendor',
|
|
||||||
'tools/wpscan-semgrep-rules',
|
|
||||||
'temp',
|
|
||||||
'tests/_data',
|
|
||||||
'tests/_output',
|
|
||||||
'tests/_support/_generated',
|
|
||||||
'vendor',
|
|
||||||
'vendor-prefixed',
|
|
||||||
'views',
|
|
||||||
];
|
|
||||||
|
|
||||||
// the "--ignore" arg takes a list of regexes, we need to anchor and escape them
|
|
||||||
$ignorePatterns = array_map(function (string $path): string {
|
|
||||||
return '^' . preg_quote(__DIR__ . DIRECTORY_SEPARATOR . $path);
|
|
||||||
}, $ignorePaths);
|
|
||||||
|
|
||||||
$stringFilesToCheck = !empty($filesToCheck) ? implode(' ', $filesToCheck) : '.';
|
|
||||||
|
|
||||||
return $this
|
|
||||||
->taskExec($task)
|
|
||||||
->arg('--ignore=' . implode(',', $ignorePatterns))
|
|
||||||
->rawArg($stringFilesToCheck)
|
|
||||||
->run();
|
|
||||||
}
|
|
||||||
|
|
||||||
public function qaFixFile($filePath) {
|
public function qaFixFile($filePath) {
|
||||||
$extension = pathinfo($filePath, PATHINFO_EXTENSION);
|
$extension = pathinfo($filePath, PATHINFO_EXTENSION);
|
||||||
if ($extension === 'php') {
|
if ($extension === 'php') {
|
||||||
|
Reference in New Issue
Block a user