yarn icon indicating copy to clipboard operation
yarn copied to clipboard

Enforcing sidedness for client package

Open natanjunges opened this issue 2 years ago • 2 comments

Implements changes proposed in #3196, starting by the least impactful ones.

natanjunges avatar Jul 02 '23 21:07 natanjunges

This looks like a good start likely should go into the next badly breaking Minecraft version.

Once this is ready we should also look into adding some tools to validate these packages. I can possibly look into moving yarn to use a split jar instead of a merged jar, should make this easier.

modmuss50 avatar Jul 03 '23 12:07 modmuss50

Where should I move the following classes to:

  • net.minecraft.client.option.ChatVisibility should go under net.minecraft.network or net.minecraft.network.message?
  • net.minecraft.client.render.BlockBreakingInfo should really go under net.minecraft.entity.player, or is there any other place where it fits better?
  • net.minecraft.client.util.NetworkUtils should go under net.minecraft.utilor net.minecraft.network?

natanjunges avatar Jul 03 '23 22:07 natanjunges