search

8p / EightPointsGuzzleBundle

⛽️ Integrates Guzzle 6.x, a PHP HTTP Client, into Symfony
MIT License
440 stars 71 forks source link

Output cUrl command #266

Closed bigfoot90 closed 5 years ago

bigfoot90 commented 5 years ago

For debugging purposes it would be nice to have a way to replicate the request out of php. I'll make a PR in the next days.

bigfoot90 commented 5 years ago

What you need to merge #267 ?

gregurco commented 5 years ago

@bigfoot90 the PR is ok. First of all I want to check some minor issue and probably to do one more fix for version 7. After that we will start preparing for v8 and your task will be merged to master, that will become v8. Are you ok, if we will merege it in few days?

bigfoot90 commented 5 years ago

This is not a BC, so it can be merged on v7 as minor version. Please let me know what can I do to speed up this and close this work. Thanks.

gregurco commented 5 years ago

@bigfoot90 it's not about BC. It's more about new release we plenned to do some time ago and to motivate developers to migrate to new version via new features. Do you need this change ASAP in release or you can use master or wait for new release? It's important for us, because you did the PR.

bigfoot90 commented 5 years ago

It's OK. It is not ASAP, I only need to close this work on my side.

gregurco commented 5 years ago

@bigfoot90, I think it's totally done on your side. You will receive emeil, when PR will be merged. Thank you!