Checking 4 spaces indent in C files in CI
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- Une demi-journée
- Accessibilité débutants
- 45/100
Piste de recherche
Examinez la configuration de CI et les sources C sous ext/openssl/, en utilisant le modèle proposé check_no_tab.sh comme point de départ. Confirmez que la vérification signale les lignes contenant des tabulations et réussit lorsqu'aucune tabulation n'est présente ; le travail est terminé lorsque la vérification s'exécute dans la CI pour les fichiers C.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Seeing the PR https://github.com/ruby/openssl/pull/973 (thanks for the PR!), I am considering if we can check if the indent is 4 spaces without tabs in CI. Right now I am seeing the following tools.
- clang-format: https://clang.llvm.org/docs/ClangFormat.html
- GNU indent: https://www.gnu.org/software/indent/
If we only check if C files don't include tabs. We may just apply the following kind of script.
$ cat check_no_tab.sh
#!/bin/bash
set -eu
if grep -rPn '\t' ext/openssl/; then
echo "FAIL"
exit 1
fi
echo "OK"
exit 0
OK case:
$ ./check_no_tab.sh
OK
Error case:
$ ./check_no_tab.sh
ext/openssl/ossl_ssl.c:3302: DefIVarID(hostname);
FAIL
What do you think?
- Langage dominant
- C
- Étoiles
- 276
- Forks
- 201
- Merge moyen
- 1 j 3 h
- PR mergées (30 j)
- 7
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ruby/openssl
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
ruby/openssl#1118 · 5 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Expose XOFs like SHAKE128 and SHAKE256Peut-être pris @rhenium l’a pris il y a 96 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
ruby/openssl#1082 · 4 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
ruby/openssl#1075 · 4 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Unchecked *_set_* callsPeut-être pris @ndossche l’a pris il y a 166 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
ruby/openssl#1038 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Add a method OpenSSL::PKey#sizePeut-être pris @herwinw l’a pris il y a 292 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
ruby/openssl#988 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de ruby/openssl
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
FujiNetWIFI/fujinet-lib#115 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
KalicoCrew/kalico#1012 ·
Les mainteneurs répondent en général sous 1 jour
-
[Fix proposed] macOS 10.9: crash on startup due to +[NSColor linkColor] (10.10+ API) in menu_macos.mmPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
joncampbell123/dosbox-x#6664 ·
Les mainteneurs répondent en général sous 1 jour
-
bug intrinsic low priority rt_lib
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour