-
Notifications
You must be signed in to change notification settings - Fork 385
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
doc(storage): describe default project search #12833
doc(storage): describe default project search #12833
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12833 +/- ##
=======================================
Coverage 93.54% 93.54%
=======================================
Files 2074 2074
Lines 180352 180352
=======================================
+ Hits 168715 168718 +3
+ Misses 11637 11634 -3
☔ View full report in Codecov by Sentry. |
google/cloud/storage/client.h
Outdated
* explicitly by the application, the returned `ListBucketsReader` will | ||
* return an error status when used. | ||
* This function will return an error if it cannot determine the "default" | ||
* project. The function will use: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"The function will use:"
Perhaps something more like "The default project is found by looking, in order, for:". Then you don't have to repeat the "If not such value is found:" each time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks.
google/cloud/storage/client.h
Outdated
* used. If neither the environment variable is set, nor a value is set | ||
* explicitly by the application, the server will reject the request and this | ||
* function will return the error status. | ||
* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra blank line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
This change is