-
Notifications
You must be signed in to change notification settings - Fork 11
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
Bump RedHat TopN to 2.4.0 #760
Conversation
I see some problematic warnings. It's better to fix them |
I created an issue and added one more warning message in there as well. I think we can go ahead and move forward, as these 2 warnings you shared should actually be fixed in the project repo, not here. |
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.
LGTM after issue. Please link issue to this PR
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.
LGTM after issue. Please link issue to be able to track the reason we open issue
No description provided.