Skip to content

[Behat] Refactor to use PHP 7.4 syntax#14038

Merged
AdamKasp merged 3 commits intoSylius:1.11from
GSadee:behat-php-7.4-syntax
Jun 1, 2022
Merged

[Behat] Refactor to use PHP 7.4 syntax#14038
AdamKasp merged 3 commits intoSylius:1.11from
GSadee:behat-php-7.4-syntax

Conversation

@GSadee
Copy link
Copy Markdown
Member

@GSadee GSadee commented May 31, 2022

Q A
Branch? 1.11
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets continuation of #12891 and #13268
License MIT

@GSadee GSadee added Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). DX Issues and PRs aimed at improving Developer eXperience. Behat Issues and PRs aimed at improving Behat usage. labels May 31, 2022
@GSadee GSadee requested a review from a team as a code owner May 31, 2022 11:33
@GSadee GSadee force-pushed the behat-php-7.4-syntax branch from 78da2b7 to eb127a4 Compare May 31, 2022 12:10
Copy link
Copy Markdown
Contributor

@Ferror Ferror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and clean

@AdamKasp AdamKasp merged commit 6d8778f into Sylius:1.11 Jun 1, 2022
@AdamKasp
Copy link
Copy Markdown
Contributor

AdamKasp commented Jun 1, 2022

Thank you, Grzegorz! 🥇

@GSadee GSadee deleted the behat-php-7.4-syntax branch June 1, 2022 08:48
GSadee added a commit that referenced this pull request Jun 2, 2022
This PR was merged into the 1.11 branch.

Discussion
----------

| Q               | A                                                            |
|-----------------|--------------------------------------------------------------|
| Branch?         | 1.11
| Bug fix?        | no                                                       |
| New feature?    | no                                                       |
| BC breaks?      | no/s                                                       |
| Deprecations?   | no
| Related tickets | based on #14038
| License         | MIT                                                          |

<!--
 - Bug fixes must be submitted against the 1.10 or 1.11 branch(the lowest possible)
 - Features and deprecations must be submitted against the master branch
 - Make sure that the correct base branch is set

 To be sure you are not breaking any Backward Compatibilities, check the documentation:
 https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->



Commits
-------

623c91c [Rector] Configure PHP version up to 8.0
Zales0123 added a commit that referenced this pull request Jun 24, 2022
This PR was merged into the 1.11 branch.

Discussion
----------

| Q               | A                                                            |
|-----------------|--------------------------------------------------------------|
| Branch?         | 1.11          |
| Bug fix?        | no                                                      |
| New feature?    | no                                                      |
| BC breaks?      | no                                                      |
| Deprecations?   | no |
| Related tickets | based on #14038, continuation of https://github.com/Sylius/Sylius/pull/13502|
| License         | MIT                                                          |

I've included here also [the PR](#13970) as it has been removed only from master, and now it was making the build failed on this PR

<!--
 - Bug fixes must be submitted against the 1.10 or 1.11 branch(the lowest possible)
 - Features and deprecations must be submitted against the master branch
 - Make sure that the correct base branch is set

 To be sure you are not breaking any Backward Compatibilities, check the documentation:
 https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html
-->


Commits
-------

3acaac2 [Behat] Introduce PHP 8.0 syntax
cb6d604 [Behat] Fix CS after introducing PHP 8.0 syntax
204e6d5 [Behat] Fix arguments order
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Behat Issues and PRs aimed at improving Behat usage. DX Issues and PRs aimed at improving Developer eXperience. Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants