Skip to content

Clean up dependencies#180

Open
FourIsTheNumber wants to merge 6 commits into
masterfrom
fix-deps
Open

Clean up dependencies#180
FourIsTheNumber wants to merge 6 commits into
masterfrom
fix-deps

Conversation

@FourIsTheNumber

@FourIsTheNumber FourIsTheNumber commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

My understanding of dependency annotations is a bit shaky, but I think this is correct.

  • Crafttweaker is only used in an optional file behind an isModLoaded check and the api would be interacted with through Zenscript, so I lowered it to compileOnly.
  • Nothing in the mod is really exposed as part of a public API except for the QED registry, so I lowered all api dependencies to implementation.
  • The TODO for ModularUI1 is very outdated, I think. I can't find any reference to the mentioned class and I tested a bunch of GUIs with the dependency removed, so I think this can be removed now.
  • Postea needs to be compileOnly.
  • I also temporarily removed Angelica as a testing dependency because GTNHLib causes constant crashes from its model system due to lack of thread safety.

@Nikolay-Sitnikov Nikolay-Sitnikov self-assigned this Jun 25, 2026
@FourIsTheNumber FourIsTheNumber mentioned this pull request Jun 25, 2026
20 tasks
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