Treat -pre version suffix as prerelease.

[dbrant]

What are you trying to accomplish?

Certain projects release packages with pre-release versions that have a suffix of -preN, which doesn't seem to get caught by dependabot as a prerelease, and gets submitted in a pull request as a stable package.

example prerelease:
https://github.com/maplibre/maplibre-native/releases/tag/android-v11.0.2-pre0

example erroneous pull request:
wikimedia/apps-android-wikipedia#4806

[abdulapopoola]

Thanks a lot @dbrant , could you please add a test and we can get this over the line :)

[dbrant]

Thanks a lot @dbrant , could you please add a test and we can get this over the line :)

whoops, of course -- how about now?

[abdulapopoola]

Merged and deployed! Thanks @dbrant !!

[deivid-rodriguez]

For what it's worth, we discussed this in the past at #6747 and decided not to deviate from what maven documents at https://maven.apache.org/pom.html#Version_Order_Specification. And -pre is not mentioned at all in there.

Funny enough, when we rejected adding "-preview", I actually mentioned "-pre" explicitly:

That said, I also feel package authors should stick to what their package manager supports, and not use their own versioning. Because now if another package author decides to name their prerelease versions as "pre", then we'll need to support that and when that stops?

[deivid-rodriguez]

Well, it was not really a clear decision, I totally understand taking the pragmatic approach over the correct, consistent one. Just saying that if we decide to do this, we may also want to reopen #6747.

[jonjanego]

Hi @dbrant - I wanted to give you heads up that we have decided to revisit this decision. See #10626

[dbrant]

@jonjanego Thanks for the note, and no worries -- it looks like we'll need to convince the MapLibre developers to adopt more standard prerelease versions instead.