Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix coding standards to fix the build #459

Merged
merged 4 commits into from
Jun 28, 2022

Conversation

Zales0123
Copy link
Member

@Zales0123 Zales0123 commented Jun 23, 2022

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

EDIT: I had to upgrade PHPStan to a more modern version (we should've done it a long time ago 馃拑) to make it work with PHP 8.1. Therefore, it resulted in a little bit more changes and skips than expected, but at least it unblocks us with the development 馃枛

@Zales0123 Zales0123 added Bug Confirmed bugs or bugfixes. DX Issues and PRs aimed at improving Developer eXperience. labels Jun 23, 2022
@Zales0123 Zales0123 requested a review from a team as a code owner June 23, 2022 08:31
@Zales0123 Zales0123 force-pushed the fix-coding-standards branch 11 times, most recently from 87cef06 to 143da71 Compare June 23, 2022 12:49
@Zales0123 Zales0123 force-pushed the fix-coding-standards branch 4 times, most recently from 1e960bb to d6f6890 Compare June 23, 2022 14:01
src/Bundle/Event/ResourceControllerEvent.php Outdated Show resolved Hide resolved
src/Bundle/Grid/Parser/OptionsParser.php Outdated Show resolved Hide resolved
src/Bundle/Storage/CookieStorage.php Outdated Show resolved Hide resolved
@Zales0123 Zales0123 merged commit 659568f into Sylius:1.10 Jun 28, 2022
@Zales0123 Zales0123 deleted the fix-coding-standards branch June 28, 2022 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Confirmed bugs or bugfixes. DX Issues and PRs aimed at improving Developer eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants