Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Hot-Fix] Fix failing specs after prophecy update#10962

Merged
pamil merged 2 commits into
Sylius:masterfrom
Zales0123:fix-build
Dec 18, 2019
Merged

[Hot-Fix] Fix failing specs after prophecy update#10962
pamil merged 2 commits into
Sylius:masterfrom
Zales0123:fix-build

Conversation

@Zales0123
Copy link
Copy Markdown
Contributor

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

It seems that these specs should never work 😄 The problem is, it's hard to specify setting a cloned object... especially if in one of this case we also have some value changed 💃 So this is obviously not a final fix, but at least it should fix a build 🚀

@Zales0123 Zales0123 added the Potential Bug Potential bugs or bugfixes, that needs to be reproduced. label Dec 18, 2019
@Zales0123 Zales0123 requested a review from a team as a code owner December 18, 2019 08:30
@pamil pamil merged commit c370da2 into Sylius:master Dec 18, 2019
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Dec 18, 2019

Thank you, Mateusz! 🥇

pamil added a commit that referenced this pull request Dec 18, 2019
This PR was merged into the 1.5 branch.

Discussion
----------

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

#10962 It should be done from this branch 💃 

Commits
-------

988e973 [Hot-Fix] Fix failing specs after prophecy update
75c5d61 Make PHPStan passing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Potential Bug Potential bugs or bugfixes, that needs to be reproduced.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants