stackabletech / stackabletech/stackable-utils
Implemente SC2044: don't use fragile loops with find
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Shell
- Sterne
- 2
- Forks
- 1
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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...
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Öffne release/create-release-tag.sh und prüfe die Schleifen in den Zeilen 142 und 160. Lies die SC2044-Referenz und führe ShellCheck für das Skript aus; abgeschlossen ist die Aufgabe, wenn beide Warnungen behoben sind, ohne die Verarbeitung von Pfaden mit Leerzeichen zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- shell
- Bereich
- release
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 52/100