You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for spotting this! An alternative solution would be to remove Guava altogether from the code base, it doesn't add many benefits (except causing issues like these :)
@rohanKanojia wdyt ? On the other hand, this is a quick fix without any side-effects asfais, so would be happy to merge it, too.
@mdxabu : Thanks, you can start by replacing Guava classes with inbuilt JDK replacements. I think for JIB functionality we do need guava dependency. Although we can remove the usage of Guava classes in our codebase.
@mdxabu : Thanks, you can start by replacing Guava classes with inbuilt JDK replacements. I think for JIB functionality we do need guava dependency. Although we can remove the usage of Guava classes in our codebase.
We only have single module for docker-maven-plugin (in the src/ folder) . Rest of the modules are examples and integration tests. You only need to make changes in files in src/ directory.
Description
We should remove Guava dependency from project .
@rohanKanojia wdyt ? On the other hand, this is a quick fix without any side-effects asfais, so would be happy to merge it, too.
Originally posted by @rhuss in #1833 (comment)
The text was updated successfully, but these errors were encountered: