Change download_as_string to download_as_bytes in gcs submodule

[alexandreyc]

Hello,

According to the documentation of download_as_string, this method is being deprecated in favor of its alias download_as_bytes.

Here is a PR that update the related code. I also updated some broken links to the official documentation.

Alexandre

[piskvorky]

Thanks Alexandre, much appreciated. CC @petedannemann – can you review please?

[petedannemann]

LGTM! Nice work. Very odd that Google decided to make this fix to download_as_string and not upload_from_string

[petedannemann]

I am confused about how coveralls is reporting that this change caused our test coverage to drop by 5.5%

[mpenkov]

I think the drop in coverage is entirely unrelated to this PR.

I'm rolling out a github actions workflow to do the building and testing. Right now it's running parallel to Travis CI. I think it may be affecting the coveralls results.

[mpenkov]

Merged! Thank you for the contribution, and congrats on your first smart_open PR @alexandreyc ! 🍾

[alexandreyc]

Thank you all!