stackabletech / stackabletech/stackable-utils

Implemente SC2044: don't use fragile loops with find

Abierto
#37 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Shell
Estrellas
2
Forks
1
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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...

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Abre release/create-release-tag.sh e inspecciona los bucles de las líneas 142 y 160. Lee la referencia de SC2044 y ejecuta ShellCheck sobre el script; la tarea estará terminada cuando ambas advertencias se hayan resuelto sin romper el manejo de rutas que contienen espacios.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
shell
Área
release
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
52/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.