QuiltMC / quiltmc.org

The source for quiltmc.org
https://quiltmc.org
Other
58 stars 44 forks source link

order changelogs based on semver #191

Closed ix0rai closed 2 months ago

ix0rai commented 3 months ago

on https://quiltmc.org/en/changelog/, versions are ordered based on the parsed markdown files instead of being ordered based on their semver number

simpledemonn commented 3 months ago

That was on purpose, it is imo more flexible that way. The project can order them as the devs think is right, even if it doesn't follow sem/flexver.

ix0rai commented 3 months ago

we could define the strategy per-project

simpledemonn commented 3 months ago

I'm honestly not a fan of hardcoding that, although I guess it could be a custom property as well. What case do you forsee where sorting the versions would actually change the order in a way that cannot be done in the original file?

ix0rai commented 3 months ago

I don't, I think it's more ergonomic to append new versions to the bottom of a changelog file instead of the top

ix0rai commented 3 months ago

although yeah, it's kinda pointless. I'll manually fix my ordering

simpledemonn commented 3 months ago

I stand corrected, sorting is absolutely needed - I changed implementation halfway through and right now it'll just depend on filesystem response order which isn't deterministic