search

PaperMC / paperweight

Gradle build system plugin for Paper and Paper forks
239 stars 47 forks source link

Add MergedProperties #226

Open jpenilla opened 12 months ago

MiniDigger commented 12 months ago

I wonder if there should be a duplicate check, if a property is defined in multiple property files, that could lead to really meh to find issues and should never be acceptable anyways?

jpenilla commented 12 months ago

Hm, probably. The more I think about this though the more I start to think we should just pull the versions out of the patches and go with a version catalog in the root project. It should be pretty trivial for forks to pull it in similar to the generated api.

(would need to check if this would cause forks to run the pipeline again after applying patches)