Skip to content

DO NOT MERGE: Logging: enum-order only synth#6460

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

DO NOT MERGE: Logging: enum-order only synth#6460
yoshi-automation wants to merge 1 commit intomasterfrom
autosynth-logging

Conversation

@yoshi-automation
Copy link
Copy Markdown
Contributor

@yoshi-automation yoshi-automation commented Nov 9, 2018

This PR was generated using Autosynth. 🌈

Here's the log from Synthtool:

synthtool > Cloning googleapis.
synthtool > Running generator for google/logging/artman_logging.yaml.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:229601142daed005bf56fbaa21a02cf7aa0b261b57677d38306f8013e536de7a
Status: Image is up to date for googleapis/artman:latest
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/python/logging-v2.
synthtool > Replaced '(^.*$\\n)*' in google/cloud/logging_v2/proto/log_entry_pb2.py.
synthtool > Replaced '(^.*$\\n)*' in google/cloud/logging_v2/proto/logging_pb2.py.
synthtool > Replaced '(^.*$\\n)*' in google/cloud/logging_v2/proto/logging_config_pb2.py.
synthtool > Replaced '(^.*$\\n)*' in google/cloud/logging_v2/proto/logging_metrics_pb2.py.
synthtool > Cleaned up 0 temporary directories.

@googlebot
Copy link
Copy Markdown

googlebot commented Nov 9, 2018

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.

What to do if you already signed the CLA

Individual signers
Corporate signers

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Nov 9, 2018
@tseaver tseaver added api: logging Issues related to the Cloud Logging API. codegen labels Nov 9, 2018
@tseaver tseaver changed the title [CHANGE ME] Re-generated logging to pick up changes in the API or client library generator. DO NOT MERGE: Logging: enum-order only synth Nov 9, 2018
@tseaver tseaver added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 9, 2018
@tseaver
Copy link
Copy Markdown
Contributor

tseaver commented Nov 9, 2018

Only changes due to googleapis/gapic-generator#2319.

@sduskis sduskis added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Nov 9, 2018
@googlebot
Copy link
Copy Markdown

googlebot commented Nov 9, 2018

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

@tseaver tseaver mentioned this pull request Nov 9, 2018
Merged
@tseaver tseaver closed this Nov 10, 2018
@tseaver tseaver deleted the autosynth-logging branch November 10, 2018 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@crwilcox crwilcox Awaiting requested review from crwilcox

Assignees

No one assigned

Labels

api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement. codegen do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yoshi-automation @googlebot @tseaver @sduskis