Skip to content

Remove GTRegistry in favour of mapped registries - #5160

Open
gustovafing wants to merge 29 commits into
1.20.1from
gus/switch-forge-registries
Open

Remove GTRegistry in favour of mapped registries#5160
gustovafing wants to merge 29 commits into
1.20.1from
gus/switch-forge-registries

Conversation

@gustovafing

Copy link
Copy Markdown
Member

What

  • Removes GTRegistry, replacing it with native MC registries.

  • No external API changes.

  • Uses a registration hack to allow registration before the register event, keeping our current registration system intact (same thing we do on 1.21)

  • No AI driven tools were used for this pull request.

How Was This Tested

Gametests pass, game starts and all content loads properly.

@gustovafing
gustovafing requested a review from a team as a code owner July 21, 2026 08:44
@gustovafing gustovafing added type: refactor Suggestion to refactor a section of code Release: Major - 0.X.0 Releases focused on Content, changes to gameplay; While maintaining mostly API stability. Ignore Version Sync Do not append this issue to the version sync issue tracker. labels Jul 21, 2026
@github-actions github-actions Bot added the Tests: Passed Game Tests have passed on this PR label Jul 21, 2026
Comment thread src/main/java/com/gregtechceu/gtceu/api/data/chemical/material/Material.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/api/data/DimensionMarker.java
Comment thread src/main/java/com/gregtechceu/gtceu/api/placeholder/PlaceholderHandler.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/common/CommonProxy.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/core/mixins/forge/GameDataMixin.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/core/mixins/BuiltInRegistriesMixin.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/data/loot/ChestGenHooks.java Outdated
Comment thread src/main/resources/gtceu.mixins.json
@github-actions github-actions Bot added Tests: Failed Game Tests have failed on this PR Tests: Passed Game Tests have passed on this PR and removed Tests: Passed Game Tests have passed on this PR Tests: Failed Game Tests have failed on this PR labels Jul 22, 2026
@gustovafing
gustovafing force-pushed the gus/switch-forge-registries branch from 2942239 to fb2b2eb Compare July 23, 2026 12:00
@github-actions github-actions Bot added Tests: Passed Game Tests have passed on this PR and removed Tests: Failed Game Tests have failed on this PR labels Jul 23, 2026
@gustovafing
gustovafing force-pushed the gus/switch-forge-registries branch from e658818 to ff22a83 Compare July 27, 2026 07:19
@github-actions github-actions Bot added Tests: Failed Game Tests have failed on this PR and removed Tests: Passed Game Tests have passed on this PR labels Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Ignore Version Sync Do not append this issue to the version sync issue tracker. Release: Major - 0.X.0 Releases focused on Content, changes to gameplay; While maintaining mostly API stability. Tests: Failed Game Tests have failed on this PR type: refactor Suggestion to refactor a section of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants