Wynntils

Wynntils

611k Downloads

ClientboundBossEventPacketAccessor is not registered in mixin configs

kristofbolyai opened this issue ยท 5 comments

commented

ClientboundBossEventPacketAccessor, which is used in FixPacketBugsFeature is not registed in wynntils.mixins.json.

@magicus, as you wrote that feature, is there a reason is or was it just a mistake?

commented

That's probably why it has regressed lately. I was wondering about that...

Let's thank loom for breaking again (that's how I discovered this). ๐Ÿ˜…

commented

Just a mistake.

Is there some way we can verify automatically that all classes in the mixin package is properly registered?

(Apart from the obvious: test the feature! ;-))

commented

That's probably why it has regressed lately. I was wondering about that...

commented

Just a mistake.

Is there some way we can verify automatically that all classes in the mixin package is properly registered?

(Apart from the obvious: test the feature! ;-))

Perhaps a custom spotless module, not sure if anyone wants to do that though.

commented

Do you wanna fix this yourself or should I make a quick PR?