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

Folia support #73

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Folia support #73

wants to merge 12 commits into from

Conversation

GeorgeV220
Copy link
Contributor

@GeorgeV220 GeorgeV220 commented Nov 4, 2024

This PR integrates the HoloEasy library with support for the Folia scheduler.

Changes Made:

  • Added dependencies for the HoloEasy core and Folia scheduler.

Implementation Details:
To use the Folia scheduler implementation, please include the following dependencies in your build.gradle.kts:

dependencies {
    implementation("com.github.GeorgeV220.holoeasy:holoeasy-core:6e6d70b6d8") {
        setTransitive(false)
    }
    implementation("com.github.GeorgeV220.holoeasy:holoeasy-folia:6e6d70b6d8") {
        setTransitive(false)
    }
}

(replace georgev22 with unlenis when you merge it for the docs)

Testing:
I recommend testing the library in your own Minecraft environment.
In Folia everything worked (using the packetevents example plugin/my own plugin)
Please confirm that the implementation works as expected before merging.

  • Library Compiles Successfully
    Ensure the library compiles without errors in the default build environment.

  • Published Successfully to JitPack
    Confirm the library can be published to JitPack without issues.

  • Resolvable from JitPack
    Test that the library can be imported as a dependency in another project through JitPack, verifying both its presence and proper version resolution.

  • Builds Without Issues in Downstream Projects
    Integrate the library into at least one downstream project to confirm it compiles and works as expected when added as a dependency.

  • Functionality Works In-Game as Expected
    Test the library in an actual game environment to ensure features function as intended, with all core features stable and without bugs.

@GeorgeV220 GeorgeV220 marked this pull request as ready for review November 4, 2024 18:54
@unldenis
Copy link
Owner

unldenis commented Nov 5, 2024

Its possibile to create two modules, a spigot and a folia ? Each depending on the holoeasy-core module

@GeorgeV220
Copy link
Contributor Author

Its possibile to create two modules, a spigot and a folia ? Each depending on the holoeasy-core module

It is, but I don't find the reason to tbh.

# Conflicts:
#	buildSrc/src/main/kotlin/buildlogic.java-conventions.gradle.kts
#	gradle/libs.versions.toml
#	holoeasy-core/src/main/kotlin/org/holoeasy/HoloEasy.kt
#	holoeasy-core/src/main/kotlin/org/holoeasy/animation/Animations.kt
#	holoeasy-core/src/main/kotlin/org/holoeasy/line/ILine.kt
#	holoeasy-core/src/main/kotlin/org/holoeasy/pool/HologramPool.kt
#	holoeasy-core/src/main/kotlin/org/holoeasy/util/BukkitFuture.kt
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