Skip to content

Fix stale shadow include group ID for emoji-java in EssentialsDiscord#6467

Merged
JRoy merged 2 commits into2.xfrom
copilot/fix-shadow-include-group-id
Mar 15, 2026
Merged

Fix stale shadow include group ID for emoji-java in EssentialsDiscord#6467
JRoy merged 2 commits into2.xfrom
copilot/fix-shadow-include-group-id

Conversation

Copy link
Contributor

Copilot AI commented Mar 15, 2026

Commit 297d0e4 migrated emoji-java from Jitpack (com.github.MinnDevelopment) to EssentialsX's own Maven repo (net.essentialsx.deps), but the include(dependency(...)) in the shadowJar block was not updated to match, leaving the shadow JAR unable to locate and bundle the dependency.

Details

Proposed fix:

  • Updated EssentialsDiscord/build.gradle shadowJar include to use the correct group ID:
    // Before
    include(dependency('com.github.MinnDevelopment:emoji-java'))
    
    // After
    include(dependency('net.essentialsx.deps:emoji-java'))

Environments tested:

OS:

Java version:

  • Most recent Paper version (1.XX.Y, git-Paper-BUILD)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

Original prompt

never updated the shadow include group id here pls fix

Reference: 297d0e4


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Co-authored-by: JRoy <10731363+JRoy@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix shadow include group ID Fix stale shadow include group ID for emoji-java in EssentialsDiscord Mar 15, 2026
Copilot AI requested a review from JRoy March 15, 2026 14:59
Copilot finished work on behalf of JRoy March 15, 2026 14:59
@JRoy JRoy marked this pull request as ready for review March 15, 2026 15:05
@JRoy JRoy enabled auto-merge March 15, 2026 15:21
@JRoy JRoy added this pull request to the merge queue Mar 15, 2026
Merged via the queue into 2.x with commit d7452bf Mar 15, 2026
2 checks passed
@JRoy JRoy deleted the copilot/fix-shadow-include-group-id branch March 15, 2026 15:52
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.

4 participants