Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix library being incorrectly declared as a LIBRARY #73

Merged
merged 2 commits into from
Jun 28, 2024
Merged

Conversation

Sychic
Copy link
Member

@Sychic Sychic commented Jun 10, 2024

Closes #70

This corrects the library being incorrectly labelled as a LIBRARY and thus breaking usage with Sinytra Connector.

LIBRARY does not have access to minecraft classes because it is in the plugin layer. GAMELIBRARY is in the game layer so it can access minecraft classes.

266791887-3cf30738-50d4-40e9-84c7-c8567d53a455
Source: SpongePowered/Sponge#3898

@Sychic Sychic requested a review from Johni0702 June 10, 2024 17:26
Sychic added a commit to EssentialGG/Elementa that referenced this pull request Jun 27, 2024
@Sychic Sychic changed the title Build: Change FMLModType to GAMELIBRARY to allow for access to minecraft classes Fix mod being incorrectly declared as a LIBRARY Jun 27, 2024
@Sychic Sychic changed the title Fix mod being incorrectly declared as a LIBRARY Fix library being incorrectly declared as a LIBRARY Jun 27, 2024
@Johni0702 Johni0702 merged commit 0696479 into master Jun 28, 2024
1 check passed
@Johni0702 Johni0702 deleted the fix/mod-type branch June 28, 2024 10:39
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.

FMLModType included in fabric jars
2 participants