Skip to content

[WIP] Fix variable interpolation in disable-all-workflows.sh - #8

Closed
mikeharder with Copilot wants to merge 2 commits into
disable-wf-macfrom
copilot/sub-pr-1
Closed

mikeharder with Copilot wants to merge 2 commits into
disable-wf-macfrom
copilot/sub-pr-1

Conversation

Copilot AI commented Feb 5, 2026 •

Copy link
Copy Markdown
Contributor
  • Update all variable references to use braces consistently (${var} instead of $var)
    • Line 11: ${#} for argument count
    • Line 12: ${0} for script name
    • Line 16: ${1} for first argument
    • Line 17: ${2} for second argument
    • Line 28: "${OWNER}"/"${REPO}" in API path
    • Line 32: "${active_ids}" in conditional check
    • Line 33: ${OWNER}/${REPO} in echo message
    • Line 37: ${active_ids} in for loop
    • Line 41: "${OWNER}"/"${REPO}"/actions/workflows/"${id}" in API path
    • Line 46: ${OWNER}/${REPO} in final echo message
  • Validate changes with shellcheck (passes with no errors)
  • Test the script functionality (no test scripts available)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: mikeharder <9459391+mikeharder@users.noreply.github.com>
@mikeharder mikeharder closed this Feb 5, 2026
Copilot AI requested a review from mikeharder February 5, 2026 21:41
Copilot stopped work on behalf of mikeharder due to an error February 5, 2026 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants