RemovePoiValidTagFix is in net.minecraft.datafixer, but all other Minecraft datafixer things are in net.minecraft.datafixers with an s at the end.
Shouldn't it be the opposite ? Package names are usually singular.
We indeed should move all of them to datafixer package instead. Yarn uses singular names unlike mojang which uses plural names (seen from their foss libraries)
Most helpful comment
Shouldn't it be the opposite ? Package names are usually singular.