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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(bigquery): Add alternative approach to setting credentials #517

Merged
merged 3 commits into from Feb 19, 2021
Merged

docs(bigquery): Add alternative approach to setting credentials #517

merged 3 commits into from Feb 19, 2021

Conversation

@alanyee
Copy link
Contributor

@alanyee alanyee commented Feb 10, 2021

Add alternative approach to setting credentials in bigquery in the comments

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 馃

@alanyee alanyee requested a review from as a code owner Feb 10, 2021
@alanyee alanyee requested review from busunkim96 and removed request for Feb 10, 2021
@google-cla google-cla bot added the cla: yes label Feb 10, 2021
@plamut
Copy link
Contributor

@plamut plamut commented Feb 11, 2021

@anguillanneuf Since this is an addition to one of the snippets, could have a quick look at it? Thanks!

Loading

@plamut plamut requested a review from anguillanneuf Feb 11, 2021
@@ -30,6 +30,11 @@ def main():
key_path, scopes=["https://www.googleapis.com/auth/cloud-platform"],
)

# Alternatively, use service_account.Credentials.from_service_account_info()
# to set credentials from a json string rather than read from a file
Copy link
Contributor

@tswast tswast Feb 16, 2021

Choose a reason for hiding this comment

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

It's not actually a string, though, right? I thought the "info" function takes a parsed value (dictionary).

Loading

Copy link
Contributor Author

@alanyee alanyee Feb 17, 2021

Choose a reason for hiding this comment

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

Right. I will correct that

Loading

Correction: json object rather than string
tswast
tswast approved these changes Feb 18, 2021
Copy link
Contributor

@tswast tswast left a comment

LGTM. Thanks!

Loading

@tswast
Copy link
Contributor

@tswast tswast commented Feb 18, 2021

Lint is failing:

nox > flake8 docs/samples
docs/samples/snippets/authenticate_service_account.py:33:81: W291 trailing whitespace
nox > Command flake8 docs/samples failed with exit code 1

Loading

samples/snippets/authenticate_service_account.py Outdated Show resolved Hide resolved
Loading
@gcf-merge-on-green
Copy link

@gcf-merge-on-green gcf-merge-on-green bot commented Feb 19, 2021

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

Loading

@plamut plamut merged commit 60fbf28 into googleapis:master Feb 19, 2021
11 of 12 checks passed
Loading
@alanyee alanyee deleted the patch-1 branch Feb 19, 2021
gcf-merge-on-green bot pushed a commit that referenced this issue Feb 25, 2021
馃 I have created a release \*beep\* \*boop\* 
---
## [2.10.0](https://www.github.com/googleapis/python-bigquery/compare/v2.9.0...v2.10.0) (2021-02-25)


### Features

* add BIGNUMERIC support ([#527](https://www.github.com/googleapis/python-bigquery/issues/527)) ([cc3394f](https://www.github.com/googleapis/python-bigquery/commit/cc3394f80934419eb00c2029bb81c92a696e7d88))


### Bug Fixes

* error using empty array of structs parameter ([#474](https://www.github.com/googleapis/python-bigquery/issues/474)) ([c1d15f4](https://www.github.com/googleapis/python-bigquery/commit/c1d15f4e5da4b7e10c00afffd59a5c7f3ded027a))
* QueryJob.exception() *returns* the errors, not raises them ([#467](https://www.github.com/googleapis/python-bigquery/issues/467)) ([d763279](https://www.github.com/googleapis/python-bigquery/commit/d7632799769248b09a8558ba18f5025ebdd9675a))


### Documentation

* **bigquery:** Add alternative approach to setting credentials ([#517](https://www.github.com/googleapis/python-bigquery/issues/517)) ([60fbf28](https://www.github.com/googleapis/python-bigquery/commit/60fbf287b0d34d5db2e61cce7a5b42735ed43d0e))
* explain retry behavior for DONE jobs ([#532](https://www.github.com/googleapis/python-bigquery/issues/532)) ([696c443](https://www.github.com/googleapis/python-bigquery/commit/696c443f0a6740be0767e12b706a7771bc1460c3))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants