rust-lang/rustfmt

Enhance diff_check with more unicode et al

Open

#5884 aperta il 13 ago 2023

Vedi su GitHub
 (13 commenti) (2 reazioni) (1 assegnatario)Rust (760 fork)batch import
E-help-wantedgood first issue

Metriche repository

Star
 (4893 star)
Metriche merge PR
 (Merge medio 10g 21h) (8 PR mergiate in 30 g)

Descrizione

Follow up action item from a topic I raised in https://github.com/rust-lang/rustfmt/pull/5864#issuecomment-1676425492 relative to adding some additional test repos to improve our coverage of string formatting with various characters and encodings.

One suggested repo to include is https://github.com/unicode-org/icu4x as well as some from https://github.com/unicode-rs

I opted to simply mention icu4x as part of this instead of directly adding it because I noticed that repo is using cargo make and I didn't have bandwidth to determine if that'd cause any issues/require any additional changes to our script.

New repos can be wired up by adding a call site with respective args here:

https://github.com/rust-lang/rustfmt/blob/b069aac44ddfdb70d55d9ae40695be44515e5bb0/ci/check_diff.sh#L170-L192

Guida contributor

Direzione di ricerca
Esamina lo script ci/check diff.sh, in particolare la sezione per aggiungere nuovi repository di test (righe 170-192). Comprendi la firma della funzione e gli argomenti richiesti. Quindi, per ogni repository proposto (es. icu4x, unicode rs), verifica la compatibilità del sistema di build (es. cargo make) e aggiungi un punto di chiamata con argomenti appropriati. Se possibile, testa localmente.
Tech stack
rustshell
Dominio
toolingtesting
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitShell scripting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65