Skip to content

Update buildscript#263

Open
DanyGames2014 wants to merge 7 commits into
ModificationStation:developfrom
DanyGames2014:update-buildscript
Open

Update buildscript#263
DanyGames2014 wants to merge 7 commits into
ModificationStation:developfrom
DanyGames2014:update-buildscript

Conversation

@DanyGames2014

Copy link
Copy Markdown
Contributor

Updates the StationAPI buildscript to latest Loom, Boom and Fabric Loader

I have tested that StationAPI (with and without test mod) runs, builds and produces the correct maven artifacts

AMI

Regarding AMI, because of a bug in Loom where it fails to properly remap references to methods from a vanilla interface implemented in a StationAPI class, I have chosen to add a mixin which adds these methods with their intermediary names as to not modify the source itself and to not cause a conflict when remapping to production. These will only get applied if the following conditions are met:

  • Is Development Environmnet
  • Always More Items is loaded
  • The enableAmiCompat program argument is entered

The program argument is automatically added by the stapi buildscript in the loom section. The program argument exists to not trigger this mixin in development environments depending on StationAPI.

@calmilamsy calmilamsy 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.

Looks fine, otherwise.

Comment thread build.gradle.kts Outdated
Comment thread build.gradle.kts Outdated
Comment thread build.gradle.kts Outdated
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.

2 participants