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

WIP: Floodgate-Fabric merge #466

Draft
wants to merge 30 commits into
base: development
Choose a base branch
from

Conversation

onebeastchris
Copy link
Member

@onebeastchris onebeastchris commented Nov 17, 2023

This is a work-in-progress PR. While i think this might be a good start, this still has quite a long way to go.

Changes made/needed:

  • Tried to split up the fabric code into fabric-platform specific code & "mod-common" code. Not yet needed, but should help with making a forge port
  • Updated gradle to 8.4
  • added architectury

TODOs:

  • Fix building
  • Re-add h2 dependency when architectury updates
  • Add modrinth publishing
  • NeoForge support - depends on architecture support for neo, cloud for neo, etc
  • Dependency configuration
  • Testing
  • Backport naming fix (GeyserMC/Floodgate-Modded@1900da2)
  • Mixin configuration
  • Universal module

and probably a few dozen other things :p

Konicai and others added 26 commits September 5, 2022 23:25
# Conflicts:
#	build-logic/build.gradle.kts
#	build.gradle.kts
#	gradle.properties
#	gradle/wrapper/gradle-wrapper.jar
#	gradle/wrapper/gradle-wrapper.properties
#	settings.gradle.kts
# Conflicts:
#	gradle.properties
#	gradle/libs.versions.toml
# Conflicts:
#	core/src/main/java/org/geysermc/floodgate/core/FloodgatePlatform.java
#	gradle.properties
Let's use architectury for easier forge support
Add floodgate.modded-conventions.gradle.kts
Fix news related package name issues
Port fabric -> mod module
Re-add AW
more buildscript shenanigans
dont create common mod isolation module
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