Skip to content

Support for enum extensions in jar generation#44

Open
lukebemish wants to merge 3 commits intoneoforged:mainfrom
lukebemish:enum-extension
Open

Support for enum extensions in jar generation#44
lukebemish wants to merge 3 commits intoneoforged:mainfrom
lukebemish:enum-extension

Conversation

@lukebemish
Copy link
Copy Markdown
Contributor

@lukebemish lukebemish commented Apr 10, 2026

Note that this impl still requires FML load the same enum extensions at runtime; this will always be a requirement in order to maintain ordering guarantees and to avoid simply reimplementing RuntimeEnumExtender in its entirety. What it gives you otherwise though is the ability to compile against the entries (together with neoforged/JavaSourceTransformer#62 and company).

Note that this impl still requires FML at runtime; it's very pared down compared to RuntimeEnumExtender
@neoforged-pr-publishing
Copy link
Copy Markdown

neoforged-pr-publishing bot commented Apr 10, 2026

  • Publish PR to GitHub Packages

Last commit published: 3344215924e3e84895f167d92a757e261402aac6 - version: 4.0.20-pr-44-enum-extension

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name = "Maven for PR #44" // https://github.com/neoforged/InstallerTools/pull/44
        url = uri("https://prmaven.neoforged.net/InstallerTools/pr44")
        content {
            includeModule("net.neoforged.installertools", "binarypatch-applier")
            includeModule("net.neoforged.installertools", "binarypatcher")
            includeModule("net.neoforged.installertools", "cli-utils")
            includeModule("net.neoforged.installertools", "installertools")
            includeModule("net.neoforged.installertools", "jarsplitter")
            includeModule("net.neoforged.installertools", "problems-api")
            includeModule("net.neoforged.installertools", "zipinject")
        }
    }
}

@lukebemish lukebemish marked this pull request as ready for review April 10, 2026 23:56
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.

1 participant