From ce16177adb1861d65ca482d6add3db1d5dbc3293 Mon Sep 17 00:00:00 2001 From: hyperpolymath <6759885+hyperpolymath@users.noreply.github.com> Date: Fri, 25 Sep 2026 20:40:44 +0000 Subject: [PATCH] chore(scripts): drop single-iteration loop flagged by shellcheck Signed-off-by: hyperpolymath <6759885+hyperpolymath@users.noreply.github.com> Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com> --- scripts/propagate-github-templates.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/propagate-github-templates.sh b/scripts/propagate-github-templates.sh index 55e87ca..34e9d96 100755 --- a/scripts/propagate-github-templates.sh +++ b/scripts/propagate-github-templates.sh @@ -144,9 +144,8 @@ $FULL. See docs/ESTATE-PROPAGATION.adoc." -m "Signed-off-by: ${GIT_AUTHOR_NAME:- --base "$(gh repo view "$FULL" --json defaultBranchRef --jq .defaultBranchRef.name)" \ --body $'## What\n\nSyncs the canonical issue forms from `hyperpolymath/aerie`:\n\n- All free-text fields are the same entry type (plain `textarea`; the `render: shell` split is gone).\n- No content field is mandatory — the only required items are the two attestation checkboxes on the bug form.\n- `{{FORGE}}`/`{{OWNER}}`/`{{REPO}}` placeholders were rendered for this repo.\n\n## Why\n\nTemplate-wide fix class (aerie issues #91/#92): reporters could not skip irrelevant fields, and one field rendered differently from its neighbours. See `docs/ESTATE-PROPAGATION.adoc` in aerie for the estate rollout plan.\n\nGenerated by `scripts/propagate-github-templates.sh`.' \ 2>&1 | tail -1 - for lab in conformance; do - gh pr edit --repo "$FULL" --add-label "$lab" >/dev/null 2>&1 || true - done + # best-effort label; the repo may not define it + gh pr edit --repo "$FULL" --add-label conformance >/dev/null 2>&1 || true ) || { rc=$? if [ "$rc" -eq 3 ]; then row "$FULL" "up to date"; else row "$FULL" "ERROR: PR step failed ($rc)"; fi