Skip to content

[CHANGE ME] Re-generated to pick up changes from self.#141

Closed
yoshi-automation wants to merge 1 commit intomasterfrom
autosynth-self
Closed

[CHANGE ME] Re-generated to pick up changes from self.#141
yoshi-automation wants to merge 1 commit intomasterfrom
autosynth-self

Conversation

@yoshi-automation
Copy link
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/4be218b6-dac7-413a-9b70-c61c140820cb/targets

  • To automatically regenerate this PR, check this box.

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 5, 2020
import google.api_core.gapic_v1.routing_header
import google.api_core.grpc_helpers
import google.api_core.page_iterator
import google.api_core.path_template
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a potentially breaking change: it relies on sombebody else to have imported it before we use it below. See: googleapis/python-storage#226

@software-dov Is this a known bug in the generator?

Choose a reason for hiding this comment

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

No, this is not a known issue in the generator. I'll file an issue to add it back under the correct circumstances.
Tracked by googleapis/gapic-generator-python#565

@tseaver tseaver added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 5, 2020
Copy link
Contributor

@tseaver tseaver left a comment

Choose a reason for hiding this comment

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

Do not merge until we resolve the dropped import issue.

tseaver added a commit that referenced this pull request Aug 19, 2020
Re-add 'google.api_core.path_template' import lost in generation.

Closes #154.
Closes #141.
Closes #142.
Closes #!43.
tseaver added a commit that referenced this pull request Aug 19, 2020
Re-add 'google.api_core.path_template' import lost in generation.

Closes #154.
Closes #141.
Closes #142.
Closes #143.
@tseaver tseaver closed this in #155 Aug 19, 2020
tseaver added a commit that referenced this pull request Aug 19, 2020
Re-add 'google.api_core.path_template' import lost in generation.

Closes #154.
Closes #141.
Closes #142.
Closes #143.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. context: none do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants