Skip to content

Conversation

@ProdPreva1l
Copy link
Contributor

New way to register a hook:

public class MyFancyHook extends Hook {
    @PluginHook(id = "MyFancyHook", register = PluginHook.Register.DELAYED, platform = "bukkit")
    public MyFancyHook(@NotNull HuskTowns plugin) {
        super(plugin);
    }
}

ProdPreva1l and others added 2 commits May 17, 2024 13:04

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link
Owner

@WiIIiam278 WiIIiam278 left a comment

Choose a reason for hiding this comment

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

top notch work, great new system! A few minor nits

@WiIIiam278
Copy link
Owner

Looks like this causes compile to fail during testing? Reflection might be breaking something here.

@ProdPreva1l
Copy link
Contributor Author

i think it has to with worldguardhook, that was the issue last time

@WiIIiam278
Copy link
Owner

it might be the Mockito trying to load the WG hook and going nuts for some reason.
is it crashing trying to load WG classes?

@ProdPreva1l
Copy link
Contributor Author

it might be the Mockito trying to load the WG hook and going nuts for some reason.

is it crashing trying to load WG classes?

i havent the slightest clue as to whats going wrong, maybe some bad accessors?

@WiIIiam278
Copy link
Owner

I did find that you're registering each hook twice

@ProdPreva1l
Copy link
Contributor Author

I did find that you're registering each hook twice

WHUUTT, lemme check this

ProdPreva1l and others added 4 commits May 26, 2024 21:13

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Removed MockBukkit.unmock(): it was indefinitely checking for task finish

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@ProdPreva1l
Copy link
Contributor Author

Bump

@ProdPreva1l
Copy link
Contributor Author

Is there reviews that i cant see or am i tweaking bc i see "1 change requested" but no actual comments

@WiIIiam278 WiIIiam278 merged commit 1833cdd into WiIIiam278:master Jun 14, 2024
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.

None yet

3 participants