Skip to content

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

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

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

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/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 <noreply@google.com>
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
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
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
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
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
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
"git": {
"name": "synthtool",
"remote": "https://github.com/googleapis/synthtool.git",
"sha": "c3caf0704f25a0c365f1c315e804a30b87c62a75"
Copy link
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
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
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
Copy link
Contributor

tseaver commented Sep 21, 2020

Superseded by #256.

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

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