-
Notifications
You must be signed in to change notification settings - Fork 224
Add Yarn support #694
Copy link
Copy link
Open
Labels
status: acceptedThis is a high-priority feature or a reproduced bugThis is a high-priority feature or a reproduced bugtype: enhancement
Metadata
Metadata
Assignees
Labels
status: acceptedThis is a high-priority feature or a reproduced bugThis is a high-priority feature or a reproduced bugtype: enhancement
Type
Fields
Give feedbackNo fields configured for issues without a type.
Yarn is the mappings that FabricMC uses to deobfuscate Minecraft. Fabric is to Yarn as Forge is to MCP.
This is distinct from adding support for Fabric itself (#663), and can wait until a bit later. Indeed, Fabric support is in my fork, while Yarn support is currently not.
There are two ways to go about this. Either generify the MCP module and instead call it the "Minecraft module" or something, or make the MCP module inherit from an abstract mappings module, which can then also be extended by a separate Yarn module. I am leaning towards the latter approach as it seems to be similar to how all the different bukkit-like platforms are supported.
This would solve issues surrounding various inspections, intentions and folding only working with MCP names.