What the developer wrote
Authored by fanquake
ci: remove TODOs from retry
TODOs are good targets for LLMs to generate PRs. Remove these TODOs,
which aren't needed, to prevent that.
What changed, and why it matters
This commit simply deletes three TODO comments from a CI helper script. There is no code behavior change, no bug fix, and no security relevance.
No security action needed. This is a non-functional cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes three TODO comments in ci/retry/retry: two noting possible dependency checks (awk, python, ruby, bc, dc) and one about escaping single quotes in arguments. No executable logic was modified. The commit message explicitly states the change is to avoid LLM-generated PRs targeting TODO comments.
Changed components
ci/retry/retryInspect captured patch +0 / −3
diff --git a/ci/retry/retry b/ci/retry/retry
index 37021f00..7942b6c4 100755
--- a/ci/retry/retry
+++ b/ci/retry/retry
@@ -4,8 +4,6 @@ __sleep_amount() {
if [ -n "$constant_sleep" ]; then
sleep_time=$constant_sleep
else
- #TODO: check for awk
- #TODO: check if user would rather use one of the other possible dependencies: python, ruby, bc, dc
sleep_time=`awk "BEGIN {t = $min_sleep * $(( (1<<($attempts -1)) )); print (t > $max_sleep ? $max_sleep : t)}"`
fi
}
@@ -43,7 +41,6 @@ retry()
P="$1"
for param in "${@:2}"; do P="$P '$param'"; done
- #TODO: replace single quotes in each arg with '"'"' ?
export RETRY_ATTEMPT=$attempts
bash -c "$P"
return_code=$?
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.