ci: merge queue runs the full main lane even for docs-only PRs (scope does not classify merge_group)
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 55/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- c, github-actions
- Ambito
- build-system, ci-cd
Direzione di ricerca
Look at the GitHub Actions workflow file that defines the 'scope' classification, likely in .github/workflows/. Understand how it currently checks for docs-only changes on pull_request events. Then examine the merge_group event context to see how the diff can be obtained against its base. The goal is to modify the scope logic to also classify merge_group candidates, so docs-only merges skip heavy lanes like macos-latest. Test by simulating a merge_group run or checking the queue's behavior after changes.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found while building the merge queue (#1264, PR #1273). It is out of that PR's scope, so it is filed here rather than folded in.
With the merge queue, every queued PR runs the FULL main lane on merge_group, including docs-only PRs. scope only classifies docs-only changes on pull_request, not on merge_group. The queue's wall time is set by macos / macos-latest running the full suite plus [99i]: about 35 min on main, against 16 on a PR (measured by the #1273 round-1 critic).
Decide: either teach scope to classify the queue candidate's diff against its base on merge_group (so docs-only merges skip the heavy lanes and still report the required checks as success), or accept the cost, since merges are a few a day. Measure the queue wall time over the first week before choosing.
- Lingua principale
- C
- Stelle
- 3
- Fork
- 7
- Merge medio
- 3h 58m
- PR unite (30g)
- 105
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 InauguralSystems/EigenScript
-
area:lint-tooling bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
InauguralSystems/EigenScript#1340 ·
I maintainer di solito rispondono entro 1 giorno
-
area:stdlib found-by:code-review kind:silent-wrong
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
InauguralSystems/EigenScript#1338 ·
I maintainer di solito rispondono entro 1 giorno
-
area:lint-tooling found-by:critic kind:docs-drift
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
InauguralSystems/EigenScript#1335 ·
I maintainer di solito rispondono entro 1 giorno
-
area:ci found-by:critic kind:gate-defect
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
InauguralSystems/EigenScript#1311 ·
I maintainer di solito rispondono entro 1 giorno
-
enrolment: decide test_gc_runner_controls.py (exempt vs enrol) and whether floors need a ratchetApertaarea:gates found-by:critic kind:decision
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
InauguralSystems/EigenScript#1280 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di InauguralSystems/EigenScript
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
NabuCasa/silabs-firmware-builder#231 · 1 commento ·
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
ClickHouse/pg_clickhouse#383 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno