rust-lang/rustfmt

Enhance diff_check with more unicode et al

Open

#5 884 ouverte le 13 août 2023

Voir sur GitHub
 (13 commentaires) (2 réactions) (1 assigné)Rust (760 forks)batch import
E-help-wantedgood first issue

Métriques du dépôt

Stars
 (4 893 stars)
Métriques de merge PR
 (Merge moyen 10j 21h) (8 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez le script ci/check diff.sh, en particulier la section ajoutant de nouveaux depots de test (lignes 170-192). Comprenez la signature de la fonction et les arguments requis. Ensuite, pour chaque depot propose (par exemple icu4x, unicode rs), verifiez la compatibilite du systeme de construction (par exemple cargo make) et ajoutez un point d'appel avec les arguments appropries. Testez localement si possible.
Stack technique
rustshell
Domaine
toolingtesting
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitShell scripting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65