Implemente SC2044: don't use fragile loops with find
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 52/100
Research direction
Open release/create-release-tag.sh and inspect the loops at lines 142 and 160. Read the SC2044 reference, then run ShellCheck on the script; done means both warnings are resolved without breaking handling of paths containing spaces.
Written by the indexing model from the issue text.
Description
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...
- Dominant language
- Shell
- Stars
- 2
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from stackabletech/stackable-utils
-
release-retro/25.3.0
Difficulty 2/5 1-3 hours Newbie friendliness 58/100
stackabletech/stackable-utils#104 ·
-
Dependency Dashboard Open
Difficulty 3/5 1-2 days Newbie friendliness 25/100
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
All issues in stackabletech/stackable-utils
Similar issues
-
Issue-Enhancement Needs-Triage
Difficulty 1/5 Under an hour Newbie friendliness 86/100
PowerShell/PowerShell#28061 · 2 reactions ·
-
Feature Request: Add ability to load custom environment variables in linux-exec-server-installer.sh Open
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
microsoft/vscode-remote-release#11867 ·
-
AuTest Bug Tests
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
apache/trafficserver#13714 ·
-
Update to NCCL 2.32 Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
conda-forge/nccl-feedstock#166 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
vllm-project/agentic-api#358 · 1 comment ·