Skip to content

Conversation

@nhojpatrick
Copy link
Contributor

Everything with this pr and PR #242 now migrated to JUnit v5, unless it depends upon junit.framework.TestCase.

@nhojpatrick nhojpatrick force-pushed the feature/junit-v5-static-imports branch from ce4c7ab to 7ae30ef Compare February 21, 2022 22:03
@nhojpatrick nhojpatrick marked this pull request as draft February 21, 2022 22:12
@nhojpatrick nhojpatrick force-pushed the feature/junit-v5-static-imports branch 3 times, most recently from 8fd83af to a396ef7 Compare February 21, 2022 23:43
@nhojpatrick nhojpatrick marked this pull request as ready for review February 21, 2022 23:43
@garydgregory
Copy link
Member

@nhojpatrick Note that the tests are faiing.

@nhojpatrick nhojpatrick marked this pull request as draft March 4, 2022 21:36
@nhojpatrick nhojpatrick force-pushed the feature/junit-v5-static-imports branch from a396ef7 to 80b677e Compare March 5, 2022 14:30
@nhojpatrick nhojpatrick marked this pull request as ready for review March 5, 2022 15:54
@nhojpatrick
Copy link
Contributor Author

@garydgregory Have removed failing tests as I couldn't quickly fix, will get in the next batch.

import junit.framework.TestResult;

public class RunTest {

Copy link
Member

@garydgregory garydgregory Mar 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the future, please don't mix in style changes, it makes the PR larger and it takes longer to review.

Copy link
Member

@garydgregory garydgregory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See 1 comment, no style changes, it's annoying to review 167 files when so many contain ONLY adding or removing blank lines.

@garydgregory garydgregory merged commit 9c49b83 into apache:master Mar 5, 2022
@nhojpatrick nhojpatrick deleted the feature/junit-v5-static-imports branch March 5, 2022 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants