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

chore: update protobuf workspace dependency to v3.13.0.#233

Closed
yoshi-automation wants to merge 1 commit into
masterfrom
autosynth-googleapis
Closed

chore: update protobuf workspace dependency to v3.13.0.#233
yoshi-automation wants to merge 1 commit into
masterfrom
autosynth-googleapis

Conversation

@yoshi-automation

Copy link
Copy Markdown
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/99f82d00-056c-44fe-a21f-a221cdf62a06/targets

  • To automatically regenerate this PR, check this box.

PiperOrigin-RevId: 327026955
Source-Link: googleapis/googleapis@0dc0a6c

PiperOrigin-RevId: 327026955

Source-Author: Google APIs <[email protected]>
Source-Date: Mon Aug 17 08:40:22 2020 -0700
Source-Repo: googleapis/googleapis
Source-Sha: 0dc0a6c0f1a9f979bc0690f0caa5fbafa3000c2c
Source-Link: googleapis/googleapis@0dc0a6c
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 18, 2020
# Generated by the protocol buffer compiler. DO NOT EDIT!
# source: google/cloud/bigquery_v2/proto/encryption_config.proto

"""Generated protocol buffer code."""

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.

Grrrr, we've got dueling synths going here, adding and removing just this one line. @software-dov, @busunkim96, any clues why?

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.

Hmm, it looks like line was removed in a manual regen. Perhaps an older version of a dependency was used there?

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.

Bigtable has similar back-and-forth, with bot-generated changes only, in https://github.com/googleapis/python-bigtable/pull/111/files

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.

The mega-generator still uses protoc, right? I'm assuming that migrating to proto-plus would be a V2 sort of thing.

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.

@tseaver Yep! It still uses protoc. proto-plus will come along with the microgen.

Maybe protobuf itself is flip-flopping on the presence of this comment between versions? This PR was supposedly triggered by a new protobuf
version googleapis/googleapis@0dc0a6c

@tseaver tseaver changed the title Update protobuf workspace dependency to v3.13.0. chore: update protobuf workspace dependency to v3.13.0. Aug 18, 2020
Comment thread synth.metadata
"git": {
"name": "synthtool",
"remote": "https://github.com/googleapis/synthtool.git",
"sha": "c3caf0704f25a0c365f1c315e804a30b87c62a75"

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.

Another oddity: why is this stanza repeated? @software-dov, @busunkim96 PTAL

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.

@SurferJeffAtGoogle Is a duplication of synthtool expected?

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.

@SurferJeffAtGoogle I can confirm that the duplication occurs here in a manual synth, and in other repositories (python-bigtable, at least, has had it doubled since 2020-06-15).

@product-auto-label product-auto-label Bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Aug 21, 2020
tseaver added a commit that referenced this pull request Sep 21, 2020
@tseaver

tseaver commented Sep 21, 2020

Copy link
Copy Markdown
Contributor

Superseded by #256.

@tseaver tseaver closed this Sep 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/python-bigquery API. cla: yes This human has signed the Contributor License Agreement. context: full

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants