Skip to content

Conversation

@unilock
Copy link
Contributor

@unilock unilock commented Dec 11, 2025

Better implementation of #793

  • Fix Create and Malum integration packs to not crash with their Forge versions
    • Create's pipe interactions will require some hackery to listen to Forge events from a Fabric mod - something similar to what I originally did for QSL, probably
  • Leave Traveler's Backpack disabled on Forge - its dependency on Forge's FluidType system cannot be easily worked around
  • Sort integration packs in code (pedantic, but this has been bothering me since forever)

All integration packs not mentioned above have been tested and working on Forge by @Shibva.

@imreallybadatnames
Copy link
Contributor

meow

@Shibva
Copy link
Contributor

Shibva commented Dec 11, 2025

Neepmeat and alloy forgery are fabric exclusive mods with no forge counterpart; it's probably fine but I'd figure I point it out like it was to me.

Give me bit to copy-paste my annotations here in git comment form from testing

@Shibva
Copy link
Contributor

Shibva commented Dec 11, 2025

// All were tested for 1.20.1 - FORGE 47.4.10 w/ FFAPI 0.92.6+1.11.14,
// Connector 1.0.0-beta.47, and ConnectorExtras-1.11.2

FARMERSDELIGHT //working with mod ver 1.20.1-1.2.9

AE2_ID // working with mod ver forge-15.4.10

GOBBER // working with mod ver 1.20.1-2.8.9

BOTANIA // working with mod ver 1.20.1-450-FORGE

Copy link
Owner

@DaFuqs DaFuqs left a comment

Choose a reason for hiding this comment

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

Looks great!

TODO: final test for classloading shenanigans on fabric

Works fine!

@DaFuqs DaFuqs merged commit 3c68a8a into DaFuqs:1.20.1-aria-for-painters Dec 16, 2025
2 checks 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.

4 participants