PaperMC / paperweight

Gradle build system plugin for Paper and Paper forks
227 stars 48 forks source link

Parchment support #136

Closed Titaniumtown closed 1 year ago

Titaniumtown commented 2 years ago

Parchment is a project that provides "Open community-sourced parameter names and javadocs" on top of Mojang's mappings. This would be really nice to have in paperweight. I don't know if this has been discussed before, so that's why I'm opening this issue.

ghost commented 2 years ago

Paperweight already supports custom parameter name mappings but it only supports tiny mappings and most likely won’t add support for Parchment mappings. An alternative is to create a parser to convert their mappings to tiny. Also adding support for javadoc mappings is a good idea that should be implemented in my opinion.

electronicboy commented 2 years ago

There was internal talks about this time ago, I believe that there was some concerns in regards to its maturity as well as the potential noise it would introduce in the meantime, etc

jpenilla commented 1 year ago

Parchment support only makes sense to add into paperweight if Paper decides to start using parchment. Actually implementing it would be trivial (as far as mappings, Javadocs are probably a non-starter for Spigot-modified files, but again this is an issue for when and if Paper wants to use Parchment). Therefore this isn't the right place for this issue.