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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 7.4 compatibility fix using implode() #81

Merged
merged 2 commits into from
Dec 10, 2020

Conversation

amarroni
Copy link
Contributor

After expected a issue with a deploy to environment we find out a compatibility problem.

@BarnyShergold
Copy link

@magento run all tests

Copy link

@BarnyShergold BarnyShergold left a comment

Choose a reason for hiding this comment

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

NIce little fix

You will need to sign the Adobe CLA - also not sure why the Travis tests are failing.

@NadiyaS
Copy link
Contributor

NadiyaS commented Nov 27, 2020

Hi @amarroni,
thank you for your contribution.
I approve this fix.
We will work on fixing failed tests. And will continue with processing your PR.

@NadiyaS
Copy link
Contributor

NadiyaS commented Nov 27, 2020

@magento-engcom-team
Copy link
Contributor

@NadiyaS the branch with code successfully imported intomagento-commerce/magento-cloud-patches repository. Branch name: imported-magento-magento-cloud-patches-81.

@NadiyaS
Copy link
Contributor

NadiyaS commented Nov 27, 2020

@magento import pr to magento-commerce/magento-cloud-patches

@magento-engcom-team
Copy link
Contributor

@NadiyaS the pull request successfully imported.

@oshmyheliuk
Copy link
Contributor

@BarnyShergold hi, after importing your PR into our private repo we did squash merge by mistake instead of regular merge, and now the author of the changes is magento-engcom-team.
Sorry about that.
If it's important for you we can revert PR and create a new one.

@BarnyShergold
Copy link

@oshmyheliuk - I didn't make the PR on this, I just reviewed it 😄

@amarroni
Copy link
Contributor Author

amarroni commented Dec 10, 2020 via email

@oshmyheliuk
Copy link
Contributor

@amarroni we reverted PR and merged again without squash on this time

magento-devops-reposync-svc pushed a commit that referenced this pull request Jul 18, 2023
MCLOUD-10736: Revert MCLOUD-10604 patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants