stackabletech / stackabletech/stackable-utils
Implemente SC2044: don't use fragile loops with find
オープン
まだ誰も着手していません。
- 主要言語
- Shell
- スター
- 2
- フォーク
- 1
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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...
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
release/create-release-tag.sh を開き、142 行目と 160 行目のループを確認してください。SC2044 のリファレンスを読み、その後スクリプトに対して ShellCheck を実行してください。パスにスペースが含まれる場合の処理を壊さずに、両方の警告が解消されれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- shell
- 領域
- release
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 52/100