Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move out nested classes from ExternalAccountCredentials #1195

Open
TimurSadykov opened this issue Apr 7, 2023 · 0 comments
Open

Move out nested classes from ExternalAccountCredentials #1195

TimurSadykov opened this issue Apr 7, 2023 · 0 comments
Assignees
@TimurSadykov
Labels
next major: breaking change this is a change that we should wait to bundle into the next major version priority: p3 Desirable enhancement or fix. May not be included in next release.

Comments

@TimurSadykov
Copy link
Member

Nested classes causes import mix up on some build environments. Right now we have full package names instead of imports as a mitigation.
@TimurSadykov TimurSadykov added the next major: breaking change this is a change that we should wait to bundle into the next major version label Apr 7, 2023
@TimurSadykov TimurSadykov mentioned this issue Apr 7, 2023
Merged
@TimurSadykov TimurSadykov self-assigned this Aug 30, 2023
@TimurSadykov TimurSadykov added the priority: p3 Desirable enhancement or fix. May not be included in next release. label Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TimurSadykov TimurSadykov

Labels
next major: breaking change this is a change that we should wait to bundle into the next major version priority: p3 Desirable enhancement or fix. May not be included in next release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TimurSadykov