Skip to content

EMI synthetic recipe warning fix#4732

Merged
gustovafing merged 1 commit intoGregTechCEu:1.20.1from
Mqrius:feature/emi-synthetic-warning-fix
Mar 14, 2026
Merged

EMI synthetic recipe warning fix#4732
gustovafing merged 1 commit intoGregTechCEu:1.20.1from
Mqrius:feature/emi-synthetic-warning-fix

Conversation

@Mqrius
Copy link
Contributor

@Mqrius Mqrius commented Mar 13, 2026

What

For custom EMI categories, the EMI recipes don't correspond to a real underlying recipe. In this case, EMI requires the path part of the recipe to start with a / to know it's not a mistake.

Right now GT registers several different synthetic recipes, some of them correctly like the Ore Vein index.

A few of them are missing the / prefix: Multiblocks, Ore Processing, and Programmed Circuits. This causes several hundred EMI errors, drowning out legitimate issues.
image

This PR adds the missing slashes.

This only affects the EMI recipe IDs for these synthetic pages, but not any actual gameplay, items, or recipes.
getId() here is specifically an EMI function, dev.emi.emi.api.recipe.EmiRecipe::getId so it doesn't affect anything else.

@Mqrius Mqrius requested a review from a team as a code owner March 13, 2026 23:28
@YoungOnionMC YoungOnionMC added type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Mar 14, 2026
@gustovafing gustovafing merged commit 7af0e91 into GregTechCEu:1.20.1 Mar 14, 2026
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants