-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-29575 Do not limit surefire to Junit 4 #7284
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
Conversation
@Apache9 This patch removes the manual config, but if needed, we can also just add the junit5 runner explicitly. |
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.
The change was done about 14 years ago, in HBASE-4781, where we bump from junit3 to junit4, so I do not think we still need keep it. Just go ahead with this PR and let's see if it works.
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit 292c9f8)
No description provided.