Skip to content

Gen lib with new generator, method signature changes#310

Merged
devchas merged 1 commit intomasterfrom
release-v4-759e2a18a5649c286b6b
Jul 23, 2020
Merged

Gen lib with new generator, method signature changes#310
devchas merged 1 commit intomasterfrom
release-v4-759e2a18a5649c286b6b

Conversation

@devchas
Copy link
Copy Markdown
Contributor

@devchas devchas commented Jul 22, 2020

No description provided.

@devchas devchas requested review from jradcliff and nwbirnie July 22, 2020 17:29
@nwbirnie
Copy link
Copy Markdown
Contributor

The release script currently fails due to b/161541273. Workaround is available in the bug. tl;dr use JDK8 for release.

@devchas devchas force-pushed the release-v4-759e2a18a5649c286b6b branch from e99a083 to 7ab5fa9 Compare July 22, 2020 19:18
@devchas devchas requested a review from nwbirnie July 22, 2020 19:18
@devchas devchas changed the title Build library with GAPIC v2 Remove overloaded method signatures Jul 22, 2020
@nwbirnie
Copy link
Copy Markdown
Contributor

I think the first line of the commit message could be more accurate here. The original message, switching to GAPICv2 covers the full scope of this change. Then we can include that, as a side-effect, we have removed some of the overloaded method signatures which were prone to cause confusion due to inclusion of boolean flags.

@devchas devchas force-pushed the release-v4-759e2a18a5649c286b6b branch from 7ab5fa9 to 2186eb8 Compare July 23, 2020 13:37
@devchas devchas changed the title Remove overloaded method signatures Gen lib w new generator, method signature changes Jul 23, 2020
The library is now generated with a new build system. A side effect of that change is that certain overloaded method signatures containing boolean flags for making requests have been removed.
@devchas devchas force-pushed the release-v4-759e2a18a5649c286b6b branch from 2186eb8 to decaf29 Compare July 23, 2020 13:54
@devchas devchas changed the title Gen lib w new generator, method signature changes Gen lib with new generator, method signature changes Jul 23, 2020
@devchas devchas merged commit b5440de into master Jul 23, 2020
@jradcliff jradcliff deleted the release-v4-759e2a18a5649c286b6b branch February 22, 2024 16:51
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