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

Allow set CurlLite option to Null #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

antonvovchenko
Copy link

Even if option not supported we can allow set it to Null.
It required for compatibility with Guzzle library - https://github.com/guzzle/guzzle/blob/master/src/Handler/CurlFactory.php#L77 .

Even if option not supported we can allow set it to Null. 
It required for compatibility with Guzzle library - https://github.com/guzzle/guzzle/blob/master/src/Handler/CurlFactory.php#L77 .
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@antonvovchenko
Copy link
Author

I signed it!

On Wed, Jul 13, 2016 at 8:21 PM, googlebot [email protected] wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll

verify. Thanks.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#6 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAVZ1slzFuUWZShcjl-P6bFo_7wjAagcks5qVR6SgaJpZM4JLprN
.

Thanks,
Anton Vovchenko

@googlebot
Copy link

CLAs look good, thanks!

zachluna added a commit that referenced this pull request Aug 5, 2021
# This is the 1st commit message:

update

# This is the commit message #2:

rgdrg

# This is the commit message #3:

Deploying to docs from  @ dc566ca 🚀

# This is the commit message #4:

Revert "Deploying to docs from  @ dc566ca 🚀"

This reverts commit 75eb9d8.

# This is the commit message #5:

Revert "Deploying to docs from  @ dc566ca 🚀"

This reverts commit 75eb9d8.

# This is the commit message #6:

Deploying to docs from  @ 263776b 🚀

# This is the commit message #7:

srfsr

# This is the commit message #8:

docs edit

# This is the commit message #9:

docs edit
zachluna added a commit that referenced this pull request Aug 5, 2021
# This is the 1st commit message:

Revert "Deploying to docs from  @ dc566ca 🚀"

This reverts commit 75eb9d8.

add docs

add docs

add docs

# This is the commit message #2:

add docs

# This is the commit message #3:

add docs

# This is the commit message #4:

permissions change

# This is the commit message #5:

permissions change

# This is the commit message #6:

update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants