Skip to content

Commit e1f5a1a

Browse files
neilimeneilime
committed
ci: add rector
Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>
1 parent 8a59655 commit e1f5a1a

File tree

12 files changed

+408
-373
lines changed

12 files changed

+408
-373
lines changed

‎.github/workflows/__shared-ci.yml‎

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,25 +24,25 @@ jobs:
2424
php-version: ${{ matrix.php-versions }}
2525
extensions: none,iconv,dom,curl,mbstring,tokenizer,xml,xmlwriter,simplexml,ctype
2626
coverage: pcov
27-
27+
2828
- name: ♻️ Get composer cache directory
2929
id: composer-cache
3030
shell: bash
3131
run: echo "dir=$(composer config cache-files-dir)" >> "$GITHUB_OUTPUT"
32-
32+
3333
- name: ♻️ Cache composer dependencies
3434
uses: actions/cache@v4
3535
with:
3636
path: ${{ steps.composer-cache.outputs.dir }}
3737
key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
3838
restore-keys: ${{ runner.os }}-composer-
39-
39+
4040
- name: ⚙️ Install dependencies
4141
shell: bash
4242
run: |
4343
composer install --no-progress --prefer-dist --optimize-autoloader
4444
composer --working-dir=tools install --no-progress --prefer-dist --optimize-autoloader
45-
45+
4646
- name: ♻️ Tools cache
4747
uses: actions/cache@v4
4848
with:
@@ -55,9 +55,14 @@ jobs:
5555
if: matrix.stable
5656
run: composer php-cs-fixer -- --format=checkstyle | tools/vendor/bin/cs2pr
5757

58+
- name: 🔬 Rector
59+
id: rector
60+
if: matrix.stable
61+
run: composer rector
62+
5863
- name: 🔬 Static analysis
5964
if: matrix.stable
60-
run: composer stan -- --error-format=checkstyle | tools/vendor/bin/cs2pr
65+
run: composer phpstan -- --error-format=github
6166

6267
- name: ♻️ Tests cache
6368
uses: actions/cache@v4
@@ -66,7 +71,7 @@ jobs:
6671
key: ${{ runner.os }}-tests-${{ github.sha }}
6772
restore-keys: |
6873
${{ runner.os }}-tests-
69-
74+
7075
- name: 🧪 Test
7176
run: composer test:ci
7277

‎Makefile‎

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,16 @@ lint: ## Execute lint for given PHP version
3434
@$(call run-php,composer php-cs-fixer $(filter-out $@,$(MAKECMDGOALS)))
3535

3636
lint-fix: ## Execute lint fixing for given PHP version
37-
@$(call run-php,composer php-cs-fixer:fix $(filter-out $@,$(MAKECMDGOALS)))
37+
@$(call run-php,composer php-cs-fixer:fix $(filter-out $@,$(MAKECMDGOALS)))
3838

39-
stan: ## Execute PHPStan for given PHP version
40-
@$(call run-php,composer stan $(filter-out $@,$(MAKECMDGOALS)))
39+
rector: ## Execute rector for given PHP version
40+
@$(call run-php,composer rector $(filter-out $@,$(MAKECMDGOALS)))
41+
42+
rector-fix: ## Execute rector fixing for given PHP version
43+
@$(call run-php,composer rector:fix $(filter-out $@,$(MAKECMDGOALS)))
44+
45+
phpstan: ## Execute PHPStan for given PHP version
46+
@$(call run-php,composer phpstan $(filter-out $@,$(MAKECMDGOALS)))
4147

4248
ci: ## Execute CI scripts for given PHP version
4349
@$(call run-php,composer ci $(filter-out $@,$(MAKECMDGOALS)))

‎README.md‎

Lines changed: 1 addition & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -44,32 +44,4 @@
4444
2. [Usage](https://neilime.github.io/php-css-lint/usage)
4545
3. [Code Coverage](https://codecov.io/gh/neilime/php-css-lint)
4646
4. [PHP Doc](https://neilime.github.io/php-css-lint/phpdoc)
47-
48-
# Development
49-
50-
## Setup
51-
52-
`PHP_VERSION` is the version of php to use during the development. Example: `8.2`
53-
54-
```sh
55-
make build-php PHP_VERSION
56-
make install PHP_VERSION
57-
```
58-
59-
## Running tests
60-
61-
```sh
62-
make test PHP_VERSION
63-
```
64-
65-
## Fix code linting
66-
67-
```sh
68-
make lint-fix PHP_VERSION
69-
```
70-
71-
## Running CI scripts
72-
73-
```sh
74-
make ci PHP_VERSION
75-
```
47+
5. [Development](https://neilime.github.io/php-css-lint/development)

‎composer.json‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,13 @@
4949
"test:ci": "@test -d pcov.enabled=1 -d max_execution_time=0 --coverage-text --coverage-clover ./build/logs/clover.xml --coverage-html ./build/coverage/",
5050
"php-cs-fixer": "@php-cs-fixer:fix --dry-run",
5151
"php-cs-fixer:fix": "tools/vendor/bin/php-cs-fixer fix --show-progress=dots --diff --config=.php-cs-fixer.dist.php",
52-
"stan": "tools/vendor/bin/phpstan analyse --level 5 src",
52+
"rector": "@rector:fix --dry-run",
53+
"rector:fix": "tools/vendor/bin/rector process src",
54+
"phpstan": "tools/vendor/bin/phpstan analyse --level max src",
5355
"ci": [
5456
"@php-cs-fixer",
55-
"@stan",
57+
"@rector",
58+
"@phpstan",
5659
"@test:ci"
5760
]
5861
},

‎rector.php‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
use Rector\Caching\ValueObject\Storage\FileCacheStorage;
6+
use Rector\Config\RectorConfig;
7+
8+
return RectorConfig::configure()
9+
->withPaths([
10+
__DIR__ . '/src',
11+
__DIR__ . '/tests',
12+
])
13+
->withPhpSets()
14+
->withAttributesSets(
15+
all: true
16+
)
17+
->withPreparedSets(
18+
deadCode: true,
19+
codeQuality: true,
20+
codingStyle: true,
21+
typeDeclarations: true,
22+
privatization: true,
23+
naming: true,
24+
instanceOf: true,
25+
earlyReturn: true,
26+
strictBooleans: true,
27+
carbon: true,
28+
rectorPreset: true,
29+
phpunitCodeQuality: true,
30+
phpunit: true,
31+
)
32+
->withCache(
33+
cacheClass: FileCacheStorage::class,
34+
cacheDirectory: __DIR__ . '/tools/cache/rector'
35+
);;

‎src/CssLint/Cli.php‎

Lines changed: 73 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -1,77 +1,88 @@
11
<?php
22

3+
declare(strict_types=1);
4+
35
namespace CssLint;
46

57
class Cli
68
{
7-
private static $SCRIPT_NAME = 'php-css-lint';
8-
private static $RETURN_CODE_ERROR = 1;
9-
private static $RETURN_CODE_SUCCESS = 0;
9+
private const SCRIPT_NAME = 'php-css-lint';
10+
11+
private const RETURN_CODE_ERROR = 1;
12+
13+
private const RETURN_CODE_SUCCESS = 0;
1014

1115
/**
1216
* Entrypoint of the cli, will execute the linter according to the given arguments
13-
* @param array $aArguments arguments to be parsed (@see $_SERVER['argv'])
17+
* @param string[] $arguments arguments to be parsed (@see $_SERVER['argv'])
1418
* @return int the return code related to the execution of the linter
1519
**/
16-
public function run(array $aArguments): int
20+
public function run(array $arguments): int
1721
{
18-
$oCliArgs = $this->parseArguments($aArguments);
19-
if (!$oCliArgs->filePathOrCssString) {
22+
$cliArgs = $this->parseArguments($arguments);
23+
if ($cliArgs->filePathOrCssString === null || $cliArgs->filePathOrCssString === '' || $cliArgs->filePathOrCssString === '0') {
2024
$this->printUsage();
21-
return self::$RETURN_CODE_SUCCESS;
25+
return self::RETURN_CODE_SUCCESS;
2226
}
2327

24-
$oProperties = new \CssLint\Properties();
25-
if ($oCliArgs->options) {
26-
$aOptions = json_decode($oCliArgs->options, true);
28+
$properties = new \CssLint\Properties();
29+
if ($cliArgs->options !== null && $cliArgs->options !== '' && $cliArgs->options !== '0') {
30+
$options = json_decode($cliArgs->options, true);
2731

28-
if (json_last_error()) {
29-
$sErrorMessage = json_last_error_msg();
30-
$this->printError('Unable to parse option argument: ' . $sErrorMessage);
31-
return self::$RETURN_CODE_ERROR;
32+
if (json_last_error() !== 0) {
33+
$errorMessage = json_last_error_msg();
34+
$this->printError('Unable to parse option argument: ' . $errorMessage);
35+
return self::RETURN_CODE_ERROR;
3236
}
3337

34-
if (!$aOptions) {
38+
if (!$options) {
3539
$this->printError('Unable to parse empty option argument');
36-
return self::$RETURN_CODE_ERROR;
40+
return self::RETURN_CODE_ERROR;
3741
}
38-
$oProperties->setOptions($aOptions);
42+
43+
if (!is_array($options)) {
44+
$this->printError('Unable to parse option argument: must be a json object');
45+
return self::RETURN_CODE_ERROR;
46+
}
47+
48+
$properties->setOptions($options);
3949
}
4050

41-
$oCssLinter = new \CssLint\Linter($oProperties);
51+
$cssLinter = new \CssLint\Linter($properties);
4252

43-
$sFilePathOrCssString = $oCliArgs->filePathOrCssString;
44-
if (!file_exists($sFilePathOrCssString)) {
45-
return $this->lintString($oCssLinter, $sFilePathOrCssString);
53+
$filePathOrCssString = $cliArgs->filePathOrCssString;
54+
if (!file_exists($filePathOrCssString)) {
55+
return $this->lintString($cssLinter, $filePathOrCssString);
4656
}
4757

48-
$sFilePath = $sFilePathOrCssString;
49-
if (!is_readable($sFilePath)) {
50-
$this->printError('File "' . $sFilePath . '" is not readable');
51-
return self::$RETURN_CODE_ERROR;
58+
$filePath = $filePathOrCssString;
59+
if (!is_readable($filePath)) {
60+
$this->printError('File "' . $filePath . '" is not readable');
61+
return self::RETURN_CODE_ERROR;
5262
}
5363

54-
return $this->lintFile($oCssLinter, $sFilePath);
64+
return $this->lintFile($cssLinter, $filePath);
5565
}
5666

5767
/**
5868
* Retrieve the parsed Cli arguments from given arguments array
69+
* @param string[] $arguments arguments to be parsed (@see $_SERVER['argv'])
5970
* @return \CssLint\CliArgs an instance of Cli arguments object containing parsed arguments
6071
*/
61-
private function parseArguments(array $aArguments): \CssLint\CliArgs
72+
private function parseArguments(array $arguments): \CssLint\CliArgs
6273
{
63-
return new \CssLint\CliArgs($aArguments);
74+
return new \CssLint\CliArgs($arguments);
6475
}
6576

6677
/**
6778
* Display usage of the cli
6879
*/
69-
private function printUsage()
80+
private function printUsage(): void
7081
{
7182
$this->printLine('Usage:' . PHP_EOL .
7283
'------' . PHP_EOL .
7384
PHP_EOL .
74-
' ' . self::$SCRIPT_NAME . ' [--options=\'{ }\'] css_file_or_string_to_lint' . PHP_EOL .
85+
' ' . self::SCRIPT_NAME . " [--options='{ }'] css_file_or_string_to_lint" . PHP_EOL .
7586
PHP_EOL .
7687
'Arguments:' . PHP_EOL .
7788
'----------' . PHP_EOL .
@@ -95,83 +106,84 @@ private function printUsage()
95106
'---------' . PHP_EOL .
96107
PHP_EOL .
97108
' Lint a CSS file:' . PHP_EOL .
98-
' ' . self::$SCRIPT_NAME . ' ./path/to/css_file_path_to_lint.css' . PHP_EOL . PHP_EOL .
109+
' ' . self::SCRIPT_NAME . ' ./path/to/css_file_path_to_lint.css' . PHP_EOL . PHP_EOL .
99110
' Lint a CSS string:' . PHP_EOL .
100-
' ' . self::$SCRIPT_NAME . ' ".test { color: red; }"' . PHP_EOL . PHP_EOL .
111+
' ' . self::SCRIPT_NAME . ' ".test { color: red; }"' . PHP_EOL . PHP_EOL .
101112
' Lint with only tabulation as indentation:' . PHP_EOL .
102-
' ' . self::$SCRIPT_NAME .
113+
' ' . self::SCRIPT_NAME .
103114
' --options=\'{ "allowedIndentationChars": ["\t"] }\' ".test { color: red; }"' . PHP_EOL .
104115
PHP_EOL . PHP_EOL);
105116
}
106117

107118
/**
108119
* Performs lint on a given file path
109-
* @param \CssLint\Linter $oCssLinter the instance of the linter
110-
* @param string $sFilePath the path of the file to be linted
120+
* @param \CssLint\Linter $cssLinter the instance of the linter
121+
* @param string $filePath the path of the file to be linted
111122
* @return int the return code related to the execution of the linter
112123
*/
113-
private function lintFile(\CssLint\Linter $oCssLinter, string $sFilePath): int
124+
private function lintFile(\CssLint\Linter $cssLinter, string $filePath): int
114125
{
115-
$this->printLine('# Lint CSS file "' . $sFilePath . '"...');
126+
$this->printLine('# Lint CSS file "' . $filePath . '"...');
116127

117-
if ($oCssLinter->lintFile($sFilePath)) {
118-
$this->printLine("\033[32m => CSS file \"" . $sFilePath . "\" is valid\033[0m" . PHP_EOL);
119-
return self::$RETURN_CODE_SUCCESS;
128+
if ($cssLinter->lintFile($filePath)) {
129+
$this->printLine("\033[32m => CSS file \"" . $filePath . "\" is valid\033[0m" . PHP_EOL);
130+
return self::RETURN_CODE_SUCCESS;
120131
}
121132

122-
$this->printLine("\033[31m => CSS file \"" . $sFilePath . "\" is not valid:\033[0m" . PHP_EOL);
123-
$this->displayLinterErrors($oCssLinter->getErrors());
124-
return self::$RETURN_CODE_ERROR;
133+
$this->printLine("\033[31m => CSS file \"" . $filePath . "\" is not valid:\033[0m" . PHP_EOL);
134+
$this->displayLinterErrors($cssLinter->getErrors());
135+
return self::RETURN_CODE_ERROR;
125136
}
126137

127138

128139
/**
129140
* Performs lint on a given string
130-
* @param \CssLint\Linter $oCssLinter the instance of the linter
131-
* @param string $sString the CSS string to be linted
141+
* @param \CssLint\Linter $cssLinter the instance of the linter
142+
* @param string $stringValue the CSS string to be linted
132143
* @return int the return code related to the execution of the linter
133144
*/
134-
private function lintString(\CssLint\Linter $oCssLinter, string $sString): int
145+
private function lintString(\CssLint\Linter $cssLinter, string $stringValue): int
135146
{
136147
$this->printLine('# Lint CSS string...');
137148

138-
if ($oCssLinter->lintString($sString)) {
149+
if ($cssLinter->lintString($stringValue)) {
139150
$this->printLine("\033[32m => CSS string is valid\033[0m" . PHP_EOL);
140-
return self::$RETURN_CODE_SUCCESS;
151+
return self::RETURN_CODE_SUCCESS;
141152
}
142153

143154
$this->printLine("\033[31m => CSS string is not valid:\033[0m" . PHP_EOL);
144-
$this->displayLinterErrors($oCssLinter->getErrors());
145-
return self::$RETURN_CODE_ERROR;
155+
$this->displayLinterErrors($cssLinter->getErrors());
156+
return self::RETURN_CODE_ERROR;
146157
}
147158

148159
/**
149160
* Display an error message
150-
* @param string $sError the message to be displayed
161+
* @param string $error the message to be displayed
151162
*/
152-
private function printError(string $sError)
163+
private function printError(string $error): void
153164
{
154-
$this->printLine("\033[31m/!\ Error: " . $sError . "\033[0m" . PHP_EOL);
165+
$this->printLine("\033[31m/!\ Error: " . $error . "\033[0m" . PHP_EOL);
155166
}
156167

157168
/**
158169
* Display the errors returned by the linter
159-
* @param array $aErrors the generated errors to be displayed
170+
* @param array $errors the generated errors to be displayed
160171
*/
161-
private function displayLinterErrors(array $aErrors)
172+
private function displayLinterErrors(array $errors): void
162173
{
163-
foreach ($aErrors as $sError) {
164-
$this->printLine("\033[31m - " . $sError . "\033[0m");
174+
foreach ($errors as $error) {
175+
$this->printLine("\033[31m - " . $error . "\033[0m");
165176
}
177+
166178
$this->printLine("");
167179
}
168180

169181
/**
170182
* Display the given message in a new line
171-
* @param string $sMessage the message to be displayed
183+
* @param string $message the message to be displayed
172184
*/
173-
private function printLine(string $sMessage)
185+
private function printLine(string $message): void
174186
{
175-
echo $sMessage . PHP_EOL;
187+
echo $message . PHP_EOL;
176188
}
177189
}

0 commit comments

Comments
 (0)