perf(compile): avoid blocking fs reads in the async compile path for the package-config check
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
Direzione di ricerca
Trova package_config_check::collect_warnings e verifica come viene chiamato dal percorso asincrono compile. Decidi se spostare la scansione limitata del file system fuori dall’executor con tokio::fs o spawn_blocking, oppure documenta perché l’I/O bloccante è accettabile. Il lavoro è completato quando una di queste opzioni è implementata senza modifiche al comportamento e i test esistenti passano.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Follow-up from the #2315 review (Rust reviewer, unposted theme).
package_config_check::collect_warnings runs on the async compile path but uses blocking std::fs (read_dir, metadata, read_to_string) to scan at most nine small files at the repository root.
The cost is negligible today, and compile does other synchronous I/O. Even so, the scan should either use tokio::fs or spawn_blocking, or document why blocking I/O is acceptable here, so the pattern doesn't spread to larger scans.
Acceptance criteria
- Either the scan runs off the async executor, or a comment justifies the bounded blocking I/O.
- No behaviour change, and the existing tests pass.
- Lingua principale
- Rust
- Stelle
- 28
- Fork
- 8
- Merge medio
- 3g 18h
- PR unite (30g)
- 10
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna 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 githubnext/ado-aw
-
bug rust
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
githubnext/ado-aw#2359 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
rust test
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
githubnext/ado-aw#2357 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
docs documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
githubnext/ado-aw#1766 ·
I maintainer di solito rispondono entro 2 giorni
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
githubnext/ado-aw#1041 ·
I maintainer di solito rispondono entro 2 giorni
-
security
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
githubnext/ado-aw#855 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di githubnext/ado-aw
Issue simili
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
I maintainer di solito rispondono entro 1 giorno
-
test(breg): postgres_startup needs the tooling feature its required-features does not declareApertaarea:breg bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
registrystack/registry-stack#2307 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
farion1231/cc-switch#8696 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
C-bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 69/100
rust-lang/rust-analyzer#23547 ·
I maintainer di solito rispondono entro 1 giorno
-
tech-debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
OpenZeppelin/guardian#570 ·
I maintainer di solito rispondono entro 1 giorno