Skip to content

[CI] Fix: ShellCheck warnings#851

Merged
gilescope merged 1 commit into
mainfrom
nacho/shell-check-workflows
Mar 3, 2026
Merged

[CI] Fix: ShellCheck warnings#851
gilescope merged 1 commit into
mainfrom
nacho/shell-check-workflows

Conversation

@NachoPal

@NachoPal NachoPal commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Overview

Fixed the ShellCheck warnings threw by: https://github.com/gilescope/vscode-shellcheck/tree/github-workflows

#851

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@NachoPal NachoPal requested a review from a team as a code owner March 3, 2026 14:14
@NachoPal NachoPal changed the title [CI} Fix: ShellCheck warnings [CI] Fix: ShellCheck warnings Mar 3, 2026
@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

kics-logo

KICS version: v2.1.19

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 47
LOW LOW 3
INFO INFO 59
TRACE TRACE 0
TOTAL TOTAL 109
Metric Values
Files scanned placeholder 27
Files parsed placeholder 27
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 11

@NachoPal NachoPal enabled auto-merge March 3, 2026 15:40
@NachoPal NachoPal added this pull request to the merge queue Mar 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 3, 2026
@NachoPal NachoPal added this pull request to the merge queue Mar 3, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 3, 2026
@gilescope gilescope added this pull request to the merge queue Mar 3, 2026
@gilescope gilescope removed this pull request from the merge queue due to a manual request Mar 3, 2026
@gilescope gilescope added this pull request to the merge queue Mar 3, 2026
Merged via the queue into main with commit 527823f Mar 3, 2026
62 of 63 checks passed
@gilescope gilescope deleted the nacho/shell-check-workflows branch March 3, 2026 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants