Skip to content

Commit

Permalink
Merge pull request #407 from cicirello/no-fat-jar
Browse files Browse the repository at this point in the history
Discontinue publishing jar-with-dependencies effective with next release
  • Loading branch information
cicirello committed May 15, 2024
2 parents 9cf6b9e + d80d1c5 commit 875c799
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 46 deletions.
1 change: 0 additions & 1 deletion .github/workflows/maven-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,5 @@ jobs:
gh release upload ${TAG} target/${{ env.artifact_name }}-${{ steps.get_version.outputs.VERSION }}.jar
gh release upload ${TAG} target/${{ env.artifact_name }}-${{ steps.get_version.outputs.VERSION }}-sources.jar
gh release upload ${TAG} target/${{ env.artifact_name }}-${{ steps.get_version.outputs.VERSION }}-javadoc.jar
gh release upload ${TAG} target/${{ env.artifact_name }}-${{ steps.get_version.outputs.VERSION }}-jar-with-dependencies.jar
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
* Fixed potential finalizer vulnerability in org.cicirello.sequences.distance.EditDistance, detected by SpotBugs analysis.

### Dependencies
* Bump rho-mu from 3.1.0 to 3.2.0
* Bump rho-mu from 3.1.0 to 4.0.0
* Bump org.cicirello:core from 2.5.0 to 2.7.0

### CI/CD
* Integrated SpotBugs static analysis into build process.
* Integrated Find Security Bugs static analysis into build process.
* Discontinued publication of a `jar-with-dependencies` (BREAKING CHANGE only if you were using the fat jar).

### Other

Expand Down
10 changes: 4 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -154,12 +154,10 @@ and (2) add the following to the repositories section of your pom.xml:

If you don't use a dependency manager that supports importing from Maven Central,
or if you simply prefer to download manually, prebuilt jars are also attached to
each [GitHub Release](https://github.com/cicirello/JavaPermutationTools).

In addition to the regular jar of the library, we also regularly publish a
`jar-with-dependencies`. The `jar-with-dependencies` does not contain any module
declarations (unlike the regular jar file which does). Therefore, the
`jar-with-dependencies` should be used on the classpath.
each [GitHub Release](https://github.com/cicirello/JavaPermutationTools). If you
manually download jar files, make sure you also get the relevant versions of the
dependencies. The simplest way to do this is to import from Maven Central, which
will obtain the relevant dependencies automatically.

## License

Expand Down
38 changes: 0 additions & 38 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -300,44 +300,6 @@
</includes>
</configuration>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>3.5.3</version>
<configuration>
<shadedArtifactAttached>true</shadedArtifactAttached>
<shadedClassifierName>jar-with-dependencies</shadedClassifierName>
<createDependencyReducedPom>false</createDependencyReducedPom>
<filters>
<filter>
<artifact>*:*</artifact>
<excludes>
<exclude>module-info.class</exclude>
</excludes>
</filter>
<filter>
<artifact>org.cicirello:core</artifact>
<excludes>
<exclude>META-INF/MANIFEST.MF</exclude>
</excludes>
</filter>
<filter>
<artifact>org.cicirello:rho-mu</artifact>
<excludes>
<exclude>META-INF/MANIFEST.MF</exclude>
</excludes>
</filter>
</filters>
</configuration>
<executions>
<execution>
<phase>package</phase>
<goals>
<goal>shade</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
<groupId>com.spotify.fmt</groupId>
<artifactId>fmt-maven-plugin</artifactId>
Expand Down

0 comments on commit 875c799

Please sign in to comment.