Skip to content

Conversation

@shmax
Copy link
Contributor

@shmax shmax commented Aug 28, 2023

Update php-cs-fixer, run on code.

@ipundit
Copy link
Contributor

ipundit commented Aug 30, 2023

You might already know about this error. This is what I get in Windows. The requirements in readme.md says PHP >= 8.1

D:\activerecord>composer style-check

php vendor/bin/php-cs-fixer fix --dry-run --verbose --diff
PHP needs to be a minimum version of PHP 5.6.0 and maximum version of PHP 8.0.*.
Current PHP version: 8.2.9.
To ignore this requirement please set PHP_CS_FIXER_IGNORE_ENV.
If you use PHP version higher than supported, you may experience code modified in a wrong way.
Please report such cases at https://github.com/FriendsOfPHP/PHP-CS-Fixer .
Script php vendor/bin/php-cs-fixer fix --dry-run --verbose --diff handling the style-check event returned with error code 1

@shmax
Copy link
Contributor Author

shmax commented Aug 31, 2023

Did you get that error on this branch?

Base automatically changed from phpstan to master September 1, 2023 05:42
@shmax
Copy link
Contributor Author

shmax commented Sep 1, 2023

Ugh, I don't understand how this works. It passes on local, but there are errors in the CLI. 😱

@shmax shmax merged commit ec20618 into master Sep 1, 2023
@shmax shmax deleted the php-cs-fixer branch September 1, 2023 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants