Skip to content

World conversion#89

Merged
FourIsTheNumber merged 14 commits into
GTNewHorizons:masterfrom
SuperSoupr:world-conversion
Jun 6, 2026
Merged

World conversion#89
FourIsTheNumber merged 14 commits into
GTNewHorizons:masterfrom
SuperSoupr:world-conversion

Conversation

@SuperSoupr

@SuperSoupr SuperSoupr commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

Finished everything currently in the mod, remaining features can be added as they get implemented.
I recommend just looking at the first commit since that has just enough to make it clear what I'm going for. I abstracted as
much of the postea and missing mapping registration as I could to make it easier to just write the transforming code.

@SuperSoupr SuperSoupr force-pushed the world-conversion branch 7 times, most recently from 0e7e8e1 to 5461fc8 Compare December 9, 2025 22:18
@SuperSoupr SuperSoupr marked this pull request as ready for review December 9, 2025 22:47
@SuperSoupr SuperSoupr changed the title World conversion draft World conversion Dec 9, 2025

@serenibyss serenibyss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Overall this is very well done, I only have a few minor comments and questions. I have not tested the specific transformations yet, but the core system looks great and I will test out each transformation in a review in the next few days

Comment thread src/main/java/com/fouristhenumber/utilitiesinexcess/compat/exu/Remappings.java Outdated
Comment thread src/main/java/com/fouristhenumber/utilitiesinexcess/compat/exu/ExuCompat.java Outdated
Comment thread src/main/java/com/fouristhenumber/utilitiesinexcess/compat/nei/NEIUtils.java Outdated
@SuperSoupr

Copy link
Copy Markdown
Contributor Author

Overall this is very well done, I only have a few minor comments and questions. I have not tested the specific transformations yet, but the core system looks great and I will test out each transformation in a review in the next few days

Thank you, I'm glad to hear the time I put into planning it paid off :)

@FourIsTheNumber

Copy link
Copy Markdown
Collaborator

I know there are some potential changes coming to postea's api that will require this to be looked at again, but will merge for now.

@FourIsTheNumber FourIsTheNumber enabled auto-merge (squash) February 15, 2026 09:20
@FourIsTheNumber FourIsTheNumber disabled auto-merge June 6, 2026 12:57
@FourIsTheNumber FourIsTheNumber enabled auto-merge (squash) June 6, 2026 13:09
@FourIsTheNumber FourIsTheNumber merged commit a95494b into GTNewHorizons:master Jun 6, 2026
1 check passed
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.

3 participants