fix(annotations): prevent the OnlyIn annotation being used on EventBusSubscriber and Mod annotated classes

[danorris709]

Adds checks to prevent the OnlyIn annotation being used in erroneous places such as classes annotated with the @Mod, or @EventBusSubscriber as outlined here

[LexManos]

Is there a real need for this? Have you seen many mods do this?
The code looks fine, a little verbose, but good enough.
If Paint wants it, he can pull it.

[PaintNinja]

Suggested change

	throw new RuntimeException("Attempted to load class "+ classNode.name + " with @Mod and @OnlyIn/@OnlyIns annotations you should instead use the side setting in the mods.toml");
	throw new RuntimeException("Found @OnlyIn on @Mod class "+ classNode.name + " - this is not allowed as it causes crashes. Remove the OnlyIn and consider setting clientSideOnly=true in the root of your mods.toml instead");

[PaintNinja]

Is there a real need for this? Have you seen many mods do this?

I added it to the wishlist because I noticed some devs do this thinking it fixes crashing servers, when in reality it just gives a different error at best. It's not common, but the less mods using OnlyIn incorrectly, the better for everyone.

The code looks fine, a little verbose, but good enough. If Paint wants it, he can pull it.

Thanks, I'll merge once my requested changes are implemented. As for the verbosity, half of it seems to be a cleanup of the RuntimeDistCleaner to use less streams