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.
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
Add servicetalk-utils-internal
module
#862
Conversation
Can one of the admins verify this patch? |
3 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
b9917f9
to
1e687e6
Compare
__Motivation__ We need to `servicetalk-utils-internal` module __Modifications__ Remove duplicate class(`MathUtil`) __Result__ Resolves apple#189
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@heowc Thank you for the contribution! Discussed approach lgtm. Few minor comments:
servicetalk-utils-internal/src/main/java/io/servicetalk/utils/internal/MathUtil.java
Outdated
Show resolved
Hide resolved
@servicetalk-bot test this please |
Thank for the review! |
@servicetalk-bot test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution! ❤️
Motivation
We need to
servicetalk-utils-internal
moduleModifications
Remove duplicate class(
MathUtil
)Result
Resolves #189