Sync bot crushing partial reviews

Aperta

#3,744 aperta il 7/8/2024 6 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 4/5
Tempo stimato: 3-5 giorni
Idoneità per principianti: 30/100

Tipo di issue: Bug
Chiarezza: Abbastanza chiara
Stato di attività: Ferma
Stack tecnologico: git, github
Ambito: devops, tooling

Direzione di ricerca

Non vengono indicati file, test o punti di ingresso. Inizia individuando la gestione dei branch e delle pull request del sync bot, quindi riproduci una sincronizzazione con una revisione parziale; il lavoro è completato quando la sincronizzazione non esegue più force-push sul lavoro sottoposto a revisione e preserva il contesto della revisione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Hi @iliakan

I discovered this in the Spanish repo,
but happens in the other repos too.
...

Previously, the bot used different branches for sync, closing the old PR and generating a new one over a new branch. Partial reviews were preserved in the closed PR.

Now, the bot closes the old PR but always uses the same branch, doing a force push on it that destroys any partial reviews.

Lingua principale: HTML
Stelle: 25.5k
Fork: 4k
Metriche di merge delle PR: Nessuna PR unita negli ultimi 30g

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Altre issue di javascript-tutorial/en.javascript.info

Proposal: Add `AbortSignal.timeout()` shorthand to "Fetch: Abort" article Aperta

Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100

javascript-tutorial/en.javascript.info#3988 · Aperta il 5/8/2026
Proposal: Add modern `Promise.allSettled` alternative to "Fetch users" task solution Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100

javascript-tutorial/en.javascript.info#3986 · Aperta il 5/8/2026
Correct Keyboard Shortcut for Console Panel Aperta

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100

javascript-tutorial/en.javascript.info#3983 · Aperta il 1/8/2026
Keyboard events: keydown filtering example can't block IME-composed input (Korean/Japanese/Chinese) Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100

javascript-tutorial/en.javascript.info#3978 · 1 commento · Aperta il 18/7/2026
Documentation improvement: clarify function assignment behavior Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100

javascript-tutorial/en.javascript.info#3973 · Aperta il 20/6/2026

Tutte le issue di javascript-tutorial/en.javascript.info

Issue simili

Add dependabot Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

TanStack/ai#1462 · Aperta il 23/9/2026
harbor-db: missing STOPSIGNAL turns every container stop into an unclean shutdown Aperta

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100

goharbor/harbor#23989 · Aperta il 23/9/2026
[oblt-aw][security] SEC-022 — findings (2026-09-23) Aperta

oblt-aw/detector/security

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

elastic/oblt-aw#2127 · 1 reazione · Aperta il 23/9/2026
[Failed pipeline] github_actions in / - Update #1587834189 Aperta

Type: AVM :a: :v: :m: Type: Bug :bug:

Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100

Azure/bicep-registry-modules#7386 · 1 commento · Aperta il 23/9/2026
Previews are built in development mode, so clicking a contributor card 404s Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

jenkins-infra/contributor-spotlight#719 · Aperta il 23/9/2026

Altre issue su DevOps