Skip to content
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

[Kitodo Core] Replace Junit 4 with Junit 5 #6093

Merged
merged 2 commits into from
Jul 31, 2024

Conversation

henning-gerhardt
Copy link
Collaborator

Part of a pull request series to replace Junit4 with Junit5 based tests. Changed made by module and are independent of the other parts.

@henning-gerhardt henning-gerhardt force-pushed the replace_junit4_with_junit5_kitodo branch from ca2447e to 9008d72 Compare June 4, 2024 09:03
@henning-gerhardt henning-gerhardt force-pushed the replace_junit4_with_junit5_kitodo branch 2 times, most recently from dcbff19 to 044a4d1 Compare June 12, 2024 06:12
@henning-gerhardt henning-gerhardt force-pushed the replace_junit4_with_junit5_kitodo branch from 044a4d1 to c3de2c7 Compare July 10, 2024 13:03
@solth solth added this to the Kitodo.Production 3.7.0 milestone Jul 17, 2024
@henning-gerhardt henning-gerhardt force-pushed the replace_junit4_with_junit5_kitodo branch 2 times, most recently from 8659e1a to 954440c Compare July 17, 2024 10:46
@henning-gerhardt henning-gerhardt force-pushed the replace_junit4_with_junit5_kitodo branch from 954440c to 40b43e9 Compare July 19, 2024 10:25
Copy link
Member

@solth solth left a comment

Choose a reason for hiding this comment

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

@henning-gerhardt Thanks a lot for this comprehensive update of the JUnit test framework! Found just one(!) little thing to nitpick! (see below)

@solth solth merged commit a690e9b into kitodo:master Jul 31, 2024
5 checks passed
@henning-gerhardt henning-gerhardt deleted the replace_junit4_with_junit5_kitodo branch July 31, 2024 07:01
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