Skip to content

chore: using full package names to avoid incorrect import merge#1193

Merged
arithmetic1728 merged 15 commits intomainfrom
stim-imports
Apr 7, 2023
Merged

chore: using full package names to avoid incorrect import merge#1193
arithmetic1728 merged 15 commits intomainfrom
stim-imports

Conversation

@TimurSadykov
Copy link
Copy Markdown

@TimurSadykov TimurSadykov commented Apr 4, 2023

Adding full package names for ExternalAccountCredentials to avoid imports mix-up due to nested classes.

Long term fix would be to move out nested classes along with next major release. Issue: googleapis/google-cloud-java#12601

@TimurSadykov TimurSadykov requested a review from a team April 4, 2023 23:54
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Apr 4, 2023
@product-auto-label product-auto-label Bot added size: u Pull request is empty. and removed size: xs Pull request size is extra small. labels Apr 4, 2023
@TimurSadykov TimurSadykov requested a review from a team April 5, 2023 22:42
@product-auto-label product-auto-label Bot added size: s Pull request size is small. and removed size: u Pull request is empty. labels Apr 5, 2023
@TimurSadykov TimurSadykov changed the title fix: moving import to fix bogus build issue fix: using full package names to fix bogus build issue Apr 7, 2023
@TimurSadykov TimurSadykov changed the title fix: using full package names to fix bogus build issue fix: using full package names to avoid incorrect import merge Apr 7, 2023
@TimurSadykov TimurSadykov changed the title fix: using full package names to avoid incorrect import merge chore: using full package names to avoid incorrect import merge Apr 7, 2023
@arithmetic1728 arithmetic1728 merged commit c52ecaf into main Apr 7, 2023
@arithmetic1728 arithmetic1728 deleted the stim-imports branch April 7, 2023 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants