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

fix: exclude 'input' from reserved names list - #788

Merged
gcf-merge-on-green[bot] merged 2 commits into
masterfrom
busunkim96-patch-2
Feb 26, 2021
Merged

fix: exclude 'input' from reserved names list#788
gcf-merge-on-green[bot] merged 2 commits into
masterfrom
busunkim96-patch-2

Conversation

@busunkim96

@busunkim96busunkim96 commented Feb 25, 2021

Copy link
Copy Markdown
Contributor

The motivation for this is to prevent a breaking change in a library that has already been made GA: texttospeech googleapis/python-texttospeech#99. The TTS protos have a field named input.

Dialogflow CX beta appears to also have a field named input, but those changes have not yet been published and it is a pre-GA API.

https://github.com/search?l=Python&p=1&q=org%3Agoogleapis+input_&type=Code

@busunkim96
busunkim96 requested a review from a teamFebruary 25, 2021 17:54
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Feb 25, 2021
@codecov

codecovBot commented Feb 25, 2021

Copy link
Copy Markdown

Codecov Report

Merging #788 (8b5d55a) into master (7c185e8) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@## master #788 +/- ##
=========================================
Coverage 100.00% 100.00% =========================================
Files 26 27 +1 Lines 1608 1623 +15 Branches 328 330 +2 =========================================
+ Hits 1608 1623 +15 
Impacted FilesCoverage Δ
gapic/utils/case.py100.00% <ø> (ø)
gapic/utils/reserved_names.py100.00% <ø> (ø)
gapic/generator/generator.py100.00% <100.00%> (ø)
gapic/schema/wrappers.py100.00% <100.00%> (ø)
gapic/utils/__init__.py100.00% <100.00%> (ø)
gapic/utils/checks.py100.00% <100.00%> (ø)

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 2e17ede...8b5d55a. Read the comment docs.

@software-dovsoftware-dov added the automerge Merge the pull request once unit tests and other checks pass. label Feb 26, 2021
@gcf-merge-on-green
gcf-merge-on-greenBot merged commit da2ff71 into masterFeb 26, 2021
@gcf-merge-on-green
gcf-merge-on-greenBot deleted the busunkim96-patch-2 branch February 26, 2021 19:16
@gcf-merge-on-greengcf-merge-on-greenBot removed the automerge Merge the pull request once unit tests and other checks pass. label Feb 26, 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