stackabletech / stackabletech/stackable-utils

Implemente SC2044: don't use fragile loops with find

Ouverte
#37 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Shell
Étoiles
2
Forks
1
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

ShellCheck flags two loops in the release script that loop over find output. This is fragile for files containing spaces.

Reference: https://www.shellcheck.net/wiki/SC2044

(stackable) ➜  stackable-utils git:(main) shellcheck release/create-release-tag.sh   

In release/create-release-tag.sh line 142:
    for file in $(find "$1/docs/modules/getting_started/examples/code" -name "*.yaml"); do
                ^-- SC2044 (warning): For loops over find output are fragile. Use find -exec or a while read loop.


In release/create-release-tag.sh line 160:
    for file in $(find "$1/docs" -name "*.adoc"); do
                ^-- SC2044 (warning): For loops over find output are fragile. Use find -exec or a while read loop.

For more information:
  https://www.shellcheck.net/wiki/SC2044 -- For loops over find output are fr...

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Ouvrez release/create-release-tag.sh et examinez les boucles aux lignes 142 et 160. Lisez la référence SC2044, puis exécutez ShellCheck sur le script ; le travail est terminé lorsque les deux avertissements sont résolus sans casser la gestion des chemins contenant des espaces.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
shell
Domaine
release
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
52/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.