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

Update Maven Configurations #1212

Merged
merged 4 commits into from
Apr 7, 2024
Merged

Update Maven Configurations #1212

merged 4 commits into from
Apr 7, 2024

Conversation

syoon2
Copy link

@syoon2 syoon2 commented Apr 5, 2024

This PR:

  • Updates the link for loading MathJax in Javadoc
    The old link was using v2.7.1, this PR bumps up to v2.7.9. Includes patches for CVE-2018-1999024 (fixed in MathJax v2.7.4).
  • Tells Maven to use the same versioning key for Surefire and Failsafe (to keep dependency versions consistent)
  • Removes the Java API detection suppression in Javadoc (first added by bafb5a1)
    The underlying cause was resolved in the upstream dependencies few versions ago, and the version JGraphT uses for build is no longer affected.
  • Adds java.version property in pom (to keep compile and javadoc plugins consistent and free of hard-coded Java version)

@jsichi
Copy link
Member

jsichi commented Apr 7, 2024

Thanks for all the maintenance work!

@jsichi jsichi merged commit ab0c45e into jgrapht:master Apr 7, 2024
3 checks passed
@syoon2 syoon2 deleted the mvn-config branch April 7, 2024 23:22
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.

None yet

2 participants