Skip to content

Commit 7f9a62d

Browse files
yaniv-golanclaude
andcommitted
ci: exclude vendored .mcp-bash/ from shellcheck and array checks
The vendored mcp-bash framework files have their own CI upstream. Our Bash 3.2 compatibility check should only cover our own scripts. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 422e0b0 commit 7f9a62d

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
# --shell=bash ensures bash-specific checks
2020
# --severity=error catches only errors (not style warnings)
2121
# Add --severity=warning for stricter checking
22-
find mcp -name '*.sh' -type f | xargs shellcheck --shell=bash --severity=error
22+
find mcp -name '*.sh' -type f -not -path 'mcp/.mcp-bash/*' | xargs shellcheck --shell=bash --severity=error
2323
2424
- name: Check Bash 3.2 array compatibility
2525
run: |
@@ -39,7 +39,7 @@ jobs:
3939
errors=0
4040
files_checked=0
4141
42-
for file in $(find mcp -name '*.sh' -type f | sort); do
42+
for file in $(find mcp -name '*.sh' -type f -not -path 'mcp/.mcp-bash/*' | sort); do
4343
# Skip if file doesn't use set -u
4444
if ! grep -qE 'set -[a-z]*u|set -o nounset' "$file" 2>/dev/null; then
4545
continue

0 commit comments

Comments
 (0)