Github - No hay nada que comparar

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! :slight_smile:

3 Me gusta

Hola,

Estamos experimentando el mismo problema. ¿Tuviste oportunidad de resolverlo?

Usando la comparación del OP (940eb28e…fc61a7c0) veo el mismo resultado. Añadiendo un carácter más del hash del segundo commit (940eb28e…fc61a7c0d) obtengo la comparación correcta.

Mi sospecha es que esto es causado por la ambigüedad del hash del commit, es decir, hay dos commits que comienzan con fc61a7c0, por lo que no sabe cuál usar. No lo veo en el historial de commits, pero podría estar huérfano.

Parece que docker_manager usa git rev-parse --short para obtener los hashes para esa URL. En teoría, eso producirá hashes únicos que comienzan con 7 (por defecto) caracteres de longitud y aumentan en longitud según sea necesario para garantizar la unicidad. Sin embargo, los commits huérfanos se ignoran al extraer, por lo que lo que git cree que es único en una instancia de Discourse podría no ser único en GitHub.

Sin embargo, esto será bastante raro y ha habido commits desde tu publicación, por lo que el problema probablemente ya haya desaparecido si vuelves a comprobar las actualizaciones.

5 Me gusta

Hola @kinetiksoft, ¡gracias por revivir este tema!
Creo que Simon tiene razón. ¡Gracias por esta increíble explicación! :slightly_smiling_face: Nunca pensé que este podría ser el problema.

Sí, el problema se resolvió para mí después de algunas confirmaciones.

2 Me gusta

Muy interesante. Gracias por averiguarlo, @Simon_Manning.

He creado una PR que actualizará docker_manager para que utilice el hash de confirmación completo en los enlaces de GitHub:

Parece que todavía no tenemos ninguna colisión de 8 caracteres en la rama main. Pero sí tenemos un puñado de colisiones de 7 caracteres:

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

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.