Tofik ♡
Tofik ♡
You have mod conflicts, judging by the logs try removing `VeryManyPlayers` and check the issue tracker if any other mods also conflict
Its better to use `Preconditions.checkState(x != null, “message”)` than requireNonNull, its even outlined in the CONTRIBUTING.md file iirc
yea that might be smarter, also what is the reason for the unnecessary assert? it should be removed if im seeing correctly
maybe it'd be better to rename the class to something like `Validations` or `ValuePreconditions`? it could also be `FloatPreconditions` or smth but not sure if there aren't more types planned...
> Not really a fan of deprecating the existing method, wouldn't that require everyone to cast to Entity to not get the deprecation warning? The existing method could also become...
We could perhaps add an `@apiNote` with a notice that their plugins might require a recompile on the next version and a link to the new overloads
> > so we shouldn't keep an overload that's basically going to do the same thing as the other one but more limited > > I don't see why not,...
done, now the `LivingEntity` overloads just redirect to the `Entity` ones
@Lulu13022002 everything should be addressed now👍 I left the repeated `to` removal tho
Returning to the vanilla world folder structure would be the best way as it would get rid of both the world migration and then the additional manual world migration should...