Github - Não há nada para comparar

[Don]

Hello,

I am on 940eb28e31 . And now if i click the commits to view it on Github.

After click it won’t find anything… Is Discourse change branch or what am i missing? I always check the commits before upgrade.

Thank you!

[kinetiksoft]

Oi,

Estamos passando pelo mesmo problema. Você teve a chance de resolvê-lo?

[Simon_Manning]

Usando a comparação do OP (940eb28e…fc61a7c0), vejo o mesmo resultado. Adicionando mais um caractere do hash do segundo commit (940eb28e…fc61a7c0d), obtenho a comparação correta.

Minha suspeita é que isso é causado pela ambiguidade do hash do commit, ou seja, existem dois commits que começam com fc61a7c0, então ele não sabe qual usar. Não vejo isso no histórico de commits, mas pode estar órfão.

Parece que o docker_manager usa git rev-parse --short para obter os hashes para essa URL. Em teoria, isso produzirá hashes únicos começando com 7 caracteres (por padrão) de comprimento e aumentando o comprimento conforme necessário para garantir a exclusividade. No entanto, commits órfãos são ignorados ao fazer o pull, então o que o git acredita ser único em uma instância do Discourse pode não ser único no GitHub.

Isso será bastante raro, no entanto, e houve commits desde a sua postagem, então o problema provavelmente já desapareceu se você verificar as atualizações novamente.

[Don]

Olá @kinetiksoft, obrigado por reativar este tópico!
Acredito que Simon está certo. Obrigado por esta explicação incrível! Eu nunca pensei que este pudesse ser o problema.

@Simon_Manning

Isso será bastante raro, no entanto, e houve commits desde a sua postagem, então o problema provavelmente já desapareceu se você verificar as atualizações novamente.

Sim, o problema foi resolvido para mim após alguns commits.

[david (David Taylor)]

Muito interessante - obrigado por descobrir isso @Simon_Manning

Eu fiz um PR que atualizará o docker_manager para que ele use o hash de commit completo nos links do GitHub:

FIX: Do not shorten commit hashes for compare links

Não parece que ainda tenhamos nenhuma colisão de 8 caracteres no branch main. Mas temos um punhado de colisões de 7 caracteres:

❯ git rev-list --all | cut -c1-7 | sort | uniq -d
27c793a
3483c7b
45a166b
96aca6d
c2e58b6
f73ed45