Skip to content

Update all test-only dependencies #4268

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.

Sign up for GitHub

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

Open
RobertBrunel wants to merge 1 commit into
base: main
Choose a base branch
from
+12 −12
Open

Update all test-only dependencies #4268

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 12 additions & 12 deletions gradle/libs.versions.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,19 +54,19 @@ slf4j = "1.7.36"

# Test-only dependency versions

assertj = "3.26.3"
apache-commonsCollections = "4.4"
assertj = "3.27.7"

@ScottDugas ScottDugas Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good: https://github.com/assertj/assertj/releases

apache-commonsCollections = "4.5.0"

@ScottDugas ScottDugas Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine: https://commons.apache.org/proper/commons-collections/changes.html#a4.5.0

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I did create: #4282

apache-commonsMath3 = "3.6.1"
apache-httpclient = "5.2.1"
bndtools = "6.3.1"
diffutils = "4.12"
hamcrest = "2.2"
jcommander = "1.81"
jline = "3.30.4"
junit = "5.14.1"
junit-platform = "1.14.1"
mockito = "3.7.7"
snakeyaml = "2.2"
apache-httpclient = "5.6.1"

@ScottDugas ScottDugas Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine: https://hc.apache.org/news.html

bndtools = "7.3.0"

@ScottDugas ScottDugas Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Release notes seem fine: https://github.com/bndtools/bnd/wiki/Changes-in-7.3.0
But this really seems like maybe it should just be removed

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I did create: #4283

diffutils = "4.17"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Did you find release notes for this.
I found: https://github.com/java-diff-utils/java-diff-utils/releases but that ends at 4.15
it looks like the commit logs imply there is a 4.17.
I went through the git log, and didn't see anything suspicious

hamcrest = "3.0"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

https://github.com/hamcrest/JavaHamcrest/releases

jcommander = "1.82"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

1.81 -. 1.82 looks fine: cbeust/jcommander@1.81...1.82
But why not go to 1.85 or 3.0?

jline = "4.1.3"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks fine: https://github.com/jline/jline3/releases

junit = "5.14.4"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine: https://docs.junit.org/5.14.4/release-notes.html

junit-platform = "1.14.4"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm guessing this corresponds to junit 5.14.4, but I couldn't find anything clearly indicating that. I see on maven central: https://central.sonatype.com/artifact/org.junit.platform/junit-platform-launcher/versions that it went from 1.14.4 to 6.0

mockito = "5.23.0"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine: https://github.com/mockito/mockito/releases

snakeyaml = "2.6"

@ScottDugas ScottDugas Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks fine: https://github.com/snakeyaml/snakeyaml/blob/master/src/changes/changes.xml

I tried following the link to their changes on their wiki, but bitbucket says I don't have access....


# Static analysis dependency versions

Expand Down
Loading