Github - Non c'è niente da confrontare

[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]

Ciao,

Stiamo riscontrando lo stesso problema. Hai avuto modo di risolverlo?

[Simon_Manning]

Usando il confronto dall’OP (940eb28e…fc61a7c0) vedo lo stesso risultato. Aggiungendo un altro carattere dell’hash del secondo commit (940eb28e…fc61a7c0d) ottengo il confronto corretto.

Il mio sospetto è che ciò sia causato dall’ambiguità dell’hash del commit, ovvero ci sono due commit che iniziano entrambi con fc61a7c0, quindi non sa quale usare. Non lo vedo nella cronologia dei commit, ma potrebbe essere orfano.

Sembra che docker_manager utilizzi git rev-parse --short per ottenere gli hash per quell’URL. In teoria, questo produrrà hash univoci a partire da 7 (per impostazione predefinita) caratteri di lunghezza e aumentandone la lunghezza se necessario per garantire l’unicità. Tuttavia, i commit orfani vengono ignorati durante il pull, quindi ciò che git ritiene univoco su un’istanza di Discourse potrebbe non essere univoco su GitHub.

Sarà comunque piuttosto raro e ci sono stati commit dopo il tuo post, quindi il problema probabilmente è già scomparso se controlli di nuovo gli aggiornamenti.

[Don]

Ciao @kinetiksoft, grazie per aver rianimato questo argomento!
Credo che Simon abbia ragione. Grazie per questa incredibile spiegazione! Non avrei mai pensato che questo potesse essere il problema.

@Simon_Manning

This will be quite rare though and there have been commits since your post so the problem has probably already gone away if you check for updates again.

Sì, il problema è risolto per me dopo alcuni commit.

[david (David Taylor)]

Molto interessante, grazie per averlo scoperto @Simon_Manning

Ho creato una PR che aggiornerà docker_manager in modo che utilizzi l’hash completo del commit nei link di GitHub:

FIX: Do not shorten commit hashes for compare links

Non sembra che abbiamo ancora collisioni di 8 caratteri sul branch main. Ma abbiamo una manciata di collisioni di 7 caratteri:

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