Skip to content

Remove Java sources from extension runtime JARs - #107

Merged
denniskuijs merged 1 commit into
mainfrom
remove-sources-from-extension-jars
Sep 29, 2026
Merged

denniskuijs merged 1 commit into
mainfrom
remove-sources-from-extension-jars

Conversation

@wborn

@wborn wborn commented Sep 25, 2026

Copy link
Copy Markdown
Member

Summary

Stop including Java source files in the normal runtime JARs for OpenRemote extensions.

The extension projects already publish dedicated sources JARs, so including sourceSets.main.allJava in the normal JAR duplicates the source files.

Changes

Remove the following from the extension modules:

jar {
  from sourceSets.main.allJava
}

The normal JARs continue to contain compiled classes and resources, while the source files remain available through the existing -sources.jar artifacts.

Motivation

This removes unnecessary duplicate content from Maven Central publications and reduces publication size without changing runtime behavior.

Checklist

  • 1. Acceptance criteria of the linked issue(s) are met
  • 2. Tests are written and all tests pass
  • 3. Changes are manually tested by you and the reviewer
  • 4. Documentation is written or updated

Remove the explicit inclusion of sourceSets.main.allJava from extension JARs.

The standard Gradle JAR already contains compiled classes and resources, while Java sources are published separately through the sources JAR.

This avoids publishing the same source files in both the runtime JAR and the sources JAR, reducing Maven Central publication size without changing runtime behavior.
@wborn
wborn requested a review from a team September 25, 2026 15:39
@denniskuijs
denniskuijs added this pull request to the merge queue Sep 29, 2026
Merged via the queue into main with commit 2940713 Sep 29, 2026
2 checks passed
@wborn
wborn deleted the remove-sources-from-extension-jars branch September 29, 2026 08:28
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.

2 participants