stackabletech / stackabletech/stackable-utils
Implemente SC2044: don't use fragile loops with find
Abierto
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
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- 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