Implemente SC2044: don't use fragile loops with find

Offen

#37 erstellt am 26.1.2023 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 52/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: shell
Bereich: release

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.

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

Vorherrschende Sprache: Shell
Sterne: 2
Forks: 1
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus stackabletech/stackable-utils

⏲ Update `version` in the demos repo `antora.yaml` Offen

release-retro/25.3.0

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 58/100

stackabletech/stackable-utils#104 · Erstellt am 24.4.2025
Dependency Dashboard Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100

stackabletech/stackable-utils#78 · Erstellt am 14.5.2024
Add tool to manage branch protection rules across many repositories Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

stackabletech/stackable-utils#66 · Erstellt am 17.4.2024

Alle Issues in stackabletech/stackable-utils