Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

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

Merged
plamut merged 3 commits intogoogleapis:masterfrom
alanyee:patch-1
Feb 19, 2021
Merged

docs(bigquery): Add alternative approach to setting credentials#517
plamut merged 3 commits intogoogleapis:masterfrom
alanyee:patch-1

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 a team as a code owner February 10, 2021 23:21
@alanyee alanyee requested review from busunkim96 and removed request for a team February 10, 2021 23:21
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Feb 10, 2021
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Feb 10, 2021
@plamut
Copy link
Contributor

plamut commented Feb 11, 2021

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

@plamut plamut requested a review from anguillanneuf February 11, 2021 06:36
)

# 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

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).

Copy link
Contributor Author

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

Correction: json object rather than string
Copy link
Contributor

@tswast tswast left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@tswast tswast added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Feb 18, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 18, 2021
@tswast
Copy link
Contributor

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

@plamut plamut added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 18, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 18, 2021
@gcf-merge-on-green
Copy link

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.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Feb 19, 2021
@plamut plamut merged commit 60fbf28 into googleapis:master Feb 19, 2021
@alanyee alanyee deleted the patch-1 branch February 19, 2021 19:51
gcf-merge-on-green bot pushed a commit that referenced this pull request Feb 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/python-bigquery API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants