Skip to content

Conversation

@unmultimedio
Copy link
Member

These additional files are imports for the requested googleapis/cloud-run module: #1092

We need to first sync these ones to at least one ref, before adding that new module.

@github-actions
Copy link

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedNov 25, 2025, 6:37 PM

@unmultimedio unmultimedio changed the title Add cloud-run requirements Add googleapis/cloud-run requirements Nov 25, 2025
Copy link
Member

@mfridman mfridman left a comment

Choose a reason for hiding this comment

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

Hope googleapis doesn't create a reverse dependency 🤞 .

Copy link
Member Author

@unmultimedio unmultimedio left a comment

Choose a reason for hiding this comment

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

Double checked imports, just to be safe.

# Required for api/client.proto
+ api/launch_stage.proto
# Required for googleapis/cloud-run
+ api/routing.proto
Copy link
Member Author

Choose a reason for hiding this comment

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

It imports only WKT:

import "google/protobuf/descriptor.proto";

+ geo/type/viewport.proto

# Required for googleapis/cloud-run
+ iam/v1/iam_policy.proto
Copy link
Member Author

Choose a reason for hiding this comment

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

It imports files that are already here, and WKT:

import "google/api/annotations.proto";
import "google/api/client.proto";
import "google/api/field_behavior.proto";
import "google/api/resource.proto";
import "google/iam/v1/options.proto";
import "google/iam/v1/policy.proto";
import "google/protobuf/field_mask.proto";


# Required for googleapis/cloud-run
+ iam/v1/iam_policy.proto
+ iam/v1/options.proto
Copy link
Member Author

Choose a reason for hiding this comment

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

Doesn't have any import.

# Required for googleapis/cloud-run
+ iam/v1/iam_policy.proto
+ iam/v1/options.proto
+ iam/v1/policy.proto
Copy link
Member Author

Choose a reason for hiding this comment

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

It imports only WKT:

import "google/type/expr.proto";

@unmultimedio unmultimedio merged commit 7caf87e into main Nov 26, 2025
6 checks passed
@unmultimedio unmultimedio deleted the jfigueroa/add-googleapis-cloud-reqs branch November 26, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants