:thumbsup:

This is technically not backward compatible (though I don't think people rely really on this).

Thanks for your reply @pingou, this is my first approach to Pagure :)

Why this approach is not backward compatible? When I did it I noticed this change would affect only how the commit hash is presented in the template. Does it affect any stored data?

It makes it so that URL slugs referencing commits with 6 characters doesn't work anymore, but I think that doesn't actually happen much in practice.

Why this approach is not backward compatible? When I did it I noticed this change would affect only how the commit hash is presented in the template. Does it affect any stored data?

Your approach is correct, the very nature of changing the default configuration
is not backward compatible. It is fine, just something to be aware of :)
(especially regarding the version of pagure that has this code change)

Hi @pingou, sorry but I did not understand if I have to change something :sweat_smile:

@josseline Your patch was in good shape, so I merged it. Thanks for the patch!