Merge pull request #20952 from nextcloud/test/noid/fix-lint-exit-code

Respect exit code of lint run - changed from -exec to xargs as this exits properly
This commit is contained in:
Morris Jobke 2020-05-13 21:42:29 +02:00 committed by GitHub
commit eec982e34b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions

View file

@ -22,6 +22,6 @@
"scripts": {
"cs:fix": "php-cs-fixer fix",
"cs:check": "php-cs-fixer fix --dry-run --diff",
"lint": "find . -name \\*.php -not -path './lib/composer/*' -not -path './build/.phan/*' -exec php -l \"{}\" \\;"
"lint": "find . -name \\*.php -not -path './lib/composer/*' -not -path './build/.phan/*' -print0 | xargs -0 -n1 php -l"
}
}

View file

@ -70,6 +70,6 @@ class AppConfig implements IAppConfig {
}
public function deleteUserValue(string $userId, string $key): void {
return $this->config->deleteUserValue($userId, $this->appName, $key);
$this->config->deleteUserValue($userId, $this->appName, $key);
}
}