vponcova added a new comment to an issue you are following:
``
I think it may be better to tie this more closely to pungi /
productmd-defined concepts like 'variant', 'subvariant',
'shortname', etc.
That is a great idea. Actually, we only need to know the product (Fedora) and the variant
(Server, Workstation). So instead of the ProductId field, we could add the Variant field
that is already specified in productmd.
Also, a pet peeve of mine, if you're going to allow - and _ in a
string field such that they may be used to concatenate things, it's really nice to
people trying to parse that field if you specifically define what they should be used for
(and somehow ensure they're not used interchangeably or ambiguously).
I agree. The text was already updated and it doesn't mention `_` anymore.
``
To reply, visit the link below or just reply to this email
https://pagure.io/releng/issue/7202