dunamai.Version.from_git() fails if 'v' prefix not set

[mtkennerly]

dunamai.Version.from_git() fails if 'v' prefix not set

[mtkennerly]

This is intentional as the default behavior, but you can set --pattern default-unprefixed (or in Python: pattern=Pattern.DefaultUnprefixed) to make the v optional. You can also pass in custom patterns without having to patch the library's constant.

This is covered in the README: https://github.com/mtkennerly/dunamai#tips

Isn't a thing it should support from scratch as PEP440 does not require it ?

The main reason is to prevent false positives from tags that weren't meant to be a version. For example, without the v, it would turn tags like 3.11-compat into version 3.11compat0.

[cclecle]

Ok thanks, but is avoiding user errors really more important than being compliant with standards... ?

• Both PEP440 ans SemVer doesn't have this prefix.
• if one is using dunamai, he would certainly use version identifier only as tag..

Would make more sense to have an option to disable it.

But anyway, the problem is not on my own code but through copier .
I will raise an issue on there side to see if they can add the corresponding option..

[mtkennerly]

Ok thanks, but is avoiding user errors really more important than being compliant with standards... ?

From my perspective, those standards apply to the final output version, but not to the tag itself. There may not be standards for tag naming, but there are at least common patterns, and I would say the v prefix is very common, even if it's not universal. Just as an anecdotal example, the Mypy and Pylint repositories use v, whereas Black and Poetry don't. There are pros and cons to both sides of this trade-off, but I do personally think that keeping the v is a good default behavior.

if one is using dunamai, he would certainly use version identifier only as tag..

I'm someone who uses Dunamai, and my projects have v in the tags 🙃 I think we would need some research first in order to say what most people do.

[cclecle]

You are right about the global usage, it is variable on projet / people.
But I thought the point of this lib was about standards, not random developers habits... That's actually why it is used on big projects isn't it ?

I see no good enougth reason to have a non-standardized format mandatory while standardized one are actually not supported by default.
Furthermore this library is advertized as 'standards-compliant version strings' (from readme).
I know it's supposed to be for writing version but again it's shame that the only accepted ones are non-standard by default for reading :-/

By the way, was not asking you if you agree to remove the 'v' bu just to make it optional.