Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

Bump junit-bom to 5.7.0 #323

Closed
wants to merge 4 commits into from
Closed

Bump junit-bom to 5.7.0 #323

wants to merge 4 commits into from

Conversation

honnix
Copy link
Member

@honnix honnix commented Oct 26, 2020

See issue here: junit-team/junit4#1678

@honnix
Copy link
Member Author

honnix commented Oct 26, 2020

This is weird:

[INFO] +- org.junit.vintage:junit-vintage-engine:jar:5.7.0:test
...
[INFO] |  \- junit:junit:jar:4.12:test

But https://mvnrepository.com/artifact/org.junit.vintage/junit-vintage-engine/5.7.0

@honnix
Copy link
Member Author

honnix commented Oct 26, 2020

It is the jackson-bom.

@honnix
Copy link
Member Author

honnix commented Oct 26, 2020

@honnix
Copy link
Member Author

honnix commented Oct 27, 2020

@patriknordlen PTAL. Not sure why coverage check complains though.

@klaraward
Copy link
Contributor

@honnix This is still relevant right? (The coverage checks are unfortunately a bit flaky)

@honnix
Copy link
Member Author

honnix commented Feb 15, 2021

@klaraward I think this is still valid since the suggestion in junit-team/junit4#1678 is to upgrade. I will need to take another look what this is really about, because it's been a while. :D

@honnix
Copy link
Member Author

honnix commented Feb 15, 2021

The coverage under java8 seems very broken and this worries me. I will take a deeper look at it.

@honnix
Copy link
Member Author

honnix commented Feb 15, 2021

spotify/styx#884 was a related change.

@klaraward klaraward changed the title Bump junit-bom Bump junit-bom to 5.7.0 Feb 16, 2021
@klaraward
Copy link
Contributor

@honnix Are you happy with us merging #336 instead?

@honnix
Copy link
Member Author

honnix commented Apr 14, 2021

Of course! Please go ahead.

@honnix honnix closed this Apr 14, 2021
@honnix honnix deleted the patch-1 branch April 14, 2021 16:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants