Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Python: Add client options. #2853

Merged
merged 11 commits into from
Jun 26, 2019
Merged

Conversation

busunkim96
Copy link
Contributor

Tests will fail until there is a new release of api-core.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 24, 2019
@busunkim96 busunkim96 added yoshi lang: python Issues specific to Python. labels Jun 24, 2019
@busunkim96 busunkim96 marked this pull request as ready for review June 25, 2019 22:31
@busunkim96
Copy link
Contributor Author

@lukesneeringer Can I have some help with the remaining failure on gapic-generator-bazel-test?

@codecov
Copy link

codecov bot commented Jun 26, 2019

Codecov Report

Merging #2853 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2853      +/-   ##
============================================
+ Coverage     87.15%   87.16%   +<.01%     
  Complexity     5791     5791              
============================================
  Files           476      476              
  Lines         22943    22944       +1     
  Branches       2476     2476              
============================================
+ Hits          19997    19998       +1     
  Misses         2111     2111              
  Partials        835      835
Impacted Files Coverage Δ Complexity Δ
...transformer/py/PythonImportSectionTransformer.java 98.96% <100%> (ø) 81 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b955cd...b66770c. Read the comment docs.

@busunkim96 busunkim96 merged commit 47863b4 into googleapis:master Jun 26, 2019
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 28, 2019
Included gapic-generator changes:

- go: remove enoding on header request param key googleapis/gapic-generator#2858
- Python: Sync 'list_*' method docstrings with implementation googleapis/gapic-generator#2857
- Python: Add client options googleapis/gapic-generator#2853
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 28, 2019
This release includes the following fixes:

- wrap all parameters in 'apostrophes' #656
- use protoc v3.9.0-rc-1 for PHP #655

Also, the following gapic-generator changes are included:

- go: remove enoding on header request param key googleapis/gapic-generator#2858
- Python: Sync 'list_*' method docstrings with implementation googleapis/gapic-generator#2857
- Python: Add client options googleapis/gapic-generator#2853
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 28, 2019
Included gapic-generator changes:

- go: remove enoding on header request param key googleapis/gapic-generator#2858
- Python: Sync 'list_*' method docstrings with implementation googleapis/gapic-generator#2857
- Python: Add client options googleapis/gapic-generator#2853
alexander-fenster added a commit to googleapis/artman that referenced this pull request Jun 28, 2019
This release includes the following fixes:

- wrap all parameters in 'apostrophes' #656
- use protoc v3.9.0-rc-1 for PHP #655

Also, the following gapic-generator changes are included:

- go: remove enoding on header request param key googleapis/gapic-generator#2858
- Python: Sync 'list_*' method docstrings with implementation googleapis/gapic-generator#2857
- Python: Add client options googleapis/gapic-generator#2853
busunkim96 added a commit to busunkim96/gapic-generator that referenced this pull request Nov 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement. lang: python Issues specific to Python. yoshi
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants