Skip to content

Remove stale Java class artifacts - #7702

Merged
mergify[bot] merged 1 commit into
modrinth:mainfrom
BreezeDelegate:fix/remove-stale-java-class-artifacts
Oct 2, 2026
Merged

mergify[bot] merged 1 commit into
modrinth:mainfrom
BreezeDelegate:fix/remove-stale-java-class-artifacts

Conversation

@BreezeDelegate

Copy link
Copy Markdown
Contributor

Removes the accidentally committed packages/app-lib/java/bin/main/**/*.class files and ignores /bin/ in that Gradle project so IDE/compiler output does not get committed again.

These class files are not used by the launcher build: packages/app-lib/build.rs invokes Gradle and packages the generated theseus.jar from java/src. The tracked classes had also drifted from the current sources; for example the tracked TheseusAgent.class contains an unresolved-compilation stub while a fresh Gradle build produces the current agent bytecode.

Validation:

  • ./gradlew clean test shadowJar --no-daemon — BUILD SUCCESSFUL
  • build/libs/theseus.jar generated successfully
  • no runtime/source code changed

@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@BreezeDelegate
BreezeDelegate force-pushed the fix/remove-stale-java-class-artifacts branch from 39fc2b2 to f593ecd Compare September 25, 2026 19:25
@Hallskii
Hallskii requested a review from aecsocket October 2, 2026 16:36
@mergify

mergify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 5 minutes 52 seconds in the queue, including 5 minutes 25 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Oct 2, 2026
@mergify mergify Bot mentioned this pull request Oct 2, 2026
6 of 10 tasks
@mergify
mergify Bot merged commit 71f35eb into modrinth:main Oct 2, 2026
7 checks passed
@mergify mergify Bot removed the queued label Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants