Skip to content

GH-1204 Move to paper-api, remove eternalcore-paper module and all associted classes and update material handling to use XMaterial.#1359

Merged
vLuckyyy merged 8 commits into
masterfrom
move-to-paper-api
Jun 7, 2026
Merged

GH-1204 Move to paper-api, remove eternalcore-paper module and all associted classes and update material handling to use XMaterial.#1359
vLuckyyy merged 8 commits into
masterfrom
move-to-paper-api

Conversation

@vLuckyyy

@vLuckyyy vLuckyyy commented Jun 7, 2026

Copy link
Copy Markdown
Member

No description provided.

…ated classes and update material handling to use `XMaterial`.
@vLuckyyy vLuckyyy requested a review from a team as a code owner June 7, 2026 17:55

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request migrates the project from Spigot API to Paper API, removing the redundant eternalcore-paper module and integrating Paper-specific features directly into the core. It also introduces XMaterial for cross-version material compatibility. While these changes modernize the codebase, several issues were identified during the review: a regression in SpawnRespawnController that ignores respawn anchors by using getBedSpawnLocation(), a potential startup crash in RandomTeleportConfig if a material is unsupported, duplicate or legacy suggestions in MaterialArgument due to XMaterial.values(), and a redundant type check in PhantomSpawnController.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@Rollczi Rollczi self-requested a review June 7, 2026 21:12
Comment thread eternalcore-core/src/main/java/com/eternalcode/core/EternalCore.java Outdated

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

Dobra ogólnie to spokoczko parę minorów i trzeba jeszcze dodać Jakiś transformer R <-> L w tym okeri żeby używało MaterialX a nie Material.valueOf

@vLuckyyy vLuckyyy linked an issue Jun 7, 2026 that may be closed by this pull request
@vLuckyyy vLuckyyy changed the title Move to paper-api, remove eternalcore-paper module and all associted classes and update material handling to use XMaterial. GH-1204 Move to paper-api, remove eternalcore-paper module and all associted classes and update material handling to use XMaterial. Jun 7, 2026
@vLuckyyy vLuckyyy merged commit 7a52e14 into master Jun 7, 2026
2 checks passed
@vLuckyyy vLuckyyy deleted the move-to-paper-api branch June 7, 2026 23:44
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.

Migrate project to full Paper API

2 participants