Skip to content

Conversation

@0utplay
Copy link
Member

@0utplay 0utplay commented Nov 10, 2024

Motivation

Due to the usage of platform inject we need to specify the name of the resulting plugin.yml. Bungeecord and nukkit lack that setting and therefore try to use the bukkit plugin.yml.

Modification

Added the pluginFileName for nukkit & bungeecord according to their plugin loaders.

Result

The cloudnet luckperms plugin works properly on these platforms.

@github-actions
Copy link

github-actions bot commented Nov 10, 2024

Test Results

 48 files  ±0   48 suites  ±0   1m 51s ⏱️ -13s
420 tests ±0  420 ✅ ±0  0 💤 ±0  0 ❌ ±0 
751 runs  ±0  751 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit a287f18. ± Comparison against base commit 7aaa1cb.

♻️ This comment has been updated with latest results.

@0utplay 0utplay self-assigned this Nov 10, 2024
@0utplay 0utplay added v: 4.X This pull should be included in the 4.0 release in: plugin An issue/pull request releated to one of the internal plugins t: fix A pull request introducing a fix for a bug. labels Nov 10, 2024
@0utplay 0utplay requested a review from derklaro November 10, 2024 13:48
@0utplay 0utplay merged commit d4ae7ff into nightly Nov 14, 2024
@0utplay 0utplay deleted the fix/luckperms-plugin-loading branch November 14, 2024 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: plugin An issue/pull request releated to one of the internal plugins t: fix A pull request introducing a fix for a bug. v: 4.X This pull should be included in the 4.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants