siom79 / japicmp

Comparison of two versions of a jar archive
https://siom79.github.io/japicmp
Apache License 2.0
712 stars 107 forks source link

Rework CI to add multi-version build and Maven repository caching #294

Closed scordio closed 2 years ago

scordio commented 3 years ago

I noticed the CI configuration was repeated three times, so I refactored it to use the build matrix.

Relevant changes:

scordio commented 2 years ago

@siom79 now that the build was triggered, it seems that japicmp has issues with Java 17+.

siom79 commented 2 years ago

@scordio Yes, this ticket addresses the problems with Java 17: #307

scordio commented 2 years ago

Now 17 passes, but further errors with 18+.

siom79 commented 2 years ago

Seems to be related to the system-rules library. I have created an issue: https://github.com/stefanbirkner/system-rules/issues/85

scordio commented 2 years ago

I noticed that there is a dedicated python script to trigger the maven-release-plugin, or at least I assume you do it locally.

Would you like to do that via GitHub workflow? I can raise a PR with something similar to https://github.com/assertj/assertj-core/blob/main/.github/workflows/release.yml.

siom79 commented 2 years ago

That would be great. I would merge it...