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

fix: consistently use _pb2 identifier - #883

Merged
gcf-merge-on-green[bot] merged 5 commits into
masterfrom
fix-pb2-refs
May 12, 2021
Merged

fix: consistently use _pb2 identifier#883
gcf-merge-on-green[bot] merged 5 commits into
masterfrom
fix-pb2-refs

Conversation

@busunkim96

@busunkim96busunkim96 commented May 12, 2021

Copy link
Copy Markdown
Contributor

Small change to fix bug noticed in googleapis/python-dialogflow#288.

The import was from google.rpc import status_pb2 but code expected gr_status

@busunkim96
busunkim96 requested a review from a teamMay 12, 2021 21:19
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label May 12, 2021
@codecov

codecovBot commented May 12, 2021

Copy link
Copy Markdown

Codecov Report

Merging #883 (142f79d) into master (7c185e8) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@## master #883 +/- ##
=========================================
Coverage 100.00% 100.00% =========================================
Files 26 27 +1 Lines 1608 1694 +86 Branches 328 347 +19 =========================================
+ Hits 1608 1694 +86 
Impacted FilesCoverage Δ
gapic/samplegen_utils/types.py100.00% <ø> (ø)
gapic/samplegen_utils/utils.py100.00% <ø> (ø)
gapic/utils/case.py100.00% <ø> (ø)
gapic/utils/reserved_names.py100.00% <ø> (ø)
gapic/generator/generator.py100.00% <100.00%> (ø)
gapic/samplegen/samplegen.py100.00% <100.00%> (ø)
gapic/schema/api.py100.00% <100.00%> (ø)
gapic/schema/metadata.py100.00% <100.00%> (ø)
gapic/schema/wrappers.py100.00% <100.00%> (ø)
gapic/utils/__init__.py100.00% <100.00%> (ø)
... and 3 more

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 7087af5...142f79d. Read the comment docs.

@software-dovsoftware-dov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you add a quick test to verify the new behavior?

@busunkim96busunkim96 added the automerge Merge the pull request once unit tests and other checks pass. label May 12, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@busunkim96@software-dov