Skip to content

Conversation

@shartte
Copy link
Collaborator

@shartte shartte commented Aug 17, 2025

No description provided.

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Oct 1, 2025
@neoforged-automation
Copy link

@shartte, this pull request has conflicts, please resolve them for this PR to move forward.


// Download version manifest
HttpRequest manifestRequest = HttpRequest.newBuilder()
.uri(URI.create("https://piston-meta.mojang.com/mc/game/version_manifest_v2.json"))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great to cache this. Imagine you are in a train and suddenly Gradle decides to re-run this task for whatever reason. That might be an argument to use NFRT :P

import java.util.zip.ZipFile;

/**
* Sourced from the userdev config json. The run templates are the only thing that we use.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment needs an update. :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase This Pull Request needs to be rebased before being merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants