Skip to content

Allow set CurlLite option to Null#6

Open
antonvovchenko wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
antonvovchenko:antonvovchenko-patch-1
Open

Allow set CurlLite option to Null#6
antonvovchenko wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
antonvovchenko:antonvovchenko-patch-1

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 notifications@github.com 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