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

feat: support custom universe domains/TPC#1212

Merged
andrewsg merged 13 commits intomainfrom
tpc-signed-url
Feb 6, 2024
Merged

feat: support custom universe domains/TPC#1212
andrewsg merged 13 commits intomainfrom
tpc-signed-url

Conversation

@andrewsg
Copy link
Copy Markdown
Contributor

No description provided.

@andrewsg andrewsg requested review from a team January 18, 2024 23:57
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: storage Issues related to the googleapis/python-storage API. labels Jan 18, 2024
@andrewsg andrewsg added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 19, 2024
@andrewsg andrewsg changed the title feat: allow custom universe domains feat: support custom universe domains/TPC Jan 19, 2024
Copy link
Copy Markdown
Contributor

@JesseLovelace JesseLovelace left a comment

Choose a reason for hiding this comment

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

Thanks Andrew!

Copy link
Copy Markdown
Contributor

@cojenco cojenco left a comment

Choose a reason for hiding this comment

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

Looks great, thanks Andrew! Just a few minor questions

Copy link
Copy Markdown

@BrennaEpp BrennaEpp left a comment

Choose a reason for hiding this comment

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

LGTM - thank you Andrew!

Please pull in googleapis/conformance-tests#89 and make required changes to your testing code to test the new cases (this can be in a separate PR).

@andrewsg andrewsg removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 6, 2024
@andrewsg andrewsg merged commit f4cf041 into main Feb 6, 2024
@andrewsg andrewsg deleted the tpc-signed-url branch February 6, 2024 21:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/python-storage API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants