Skip to content

build(owlbot): un-final static create methods in Admin Base clients#2898

Open
jinseopkim0 wants to merge 2 commits intomainfrom
postprocessor
Open

build(owlbot): un-final static create methods in Admin Base clients#2898
jinseopkim0 wants to merge 2 commits intomainfrom
postprocessor

Conversation

@jinseopkim0
Copy link
Copy Markdown
Contributor

Updates owlbot.py to programmatically remove the `final` modifier from the static `create` factory methods in the generated BaseBigtableTableAdminClient and BaseBigtableInstanceAdminClient.

b/502616786

Updates owlbot.py to programmatically remove the \`final\` modifier
from the static \`create\` factory methods in the generated
BaseBigtableTableAdminClient and BaseBigtableInstanceAdminClient.
@product-auto-label product-auto-label Bot added size: xs Pull request size is extra small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Apr 21, 2026
@jinseopkim0 jinseopkim0 marked this pull request as ready for review April 21, 2026 19:44
@jinseopkim0 jinseopkim0 requested review from a team as code owners April 21, 2026 19:44
@jinseopkim0 jinseopkim0 requested review from blakeli0 and lqiu96 April 21, 2026 19:44
@product-auto-label product-auto-label Bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Apr 21, 2026
Comment thread owlbot.py
s.replace(
f"{library}/**/BaseBigtable*AdminClient.java",
r"public static final BaseBigtable(.*)AdminClient create\(",
r"public static BaseBigtable\1AdminClient create("
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.

Can we remove public as well if they are supposed to be overridden?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants