Skip to content

fix: base template build fails on vendor/.keep gitignore negation#1994

Merged
bpamiri merged 1 commit intodevelopfrom
peter/fix-build-vendor-keep
Mar 26, 2026
Merged

fix: base template build fails on vendor/.keep gitignore negation#1994
bpamiri merged 1 commit intodevelopfrom
peter/fix-build-vendor-keep

Conversation

@bpamiri
Copy link
Copy Markdown
Collaborator

@bpamiri bpamiri commented Mar 26, 2026

Summary

  • The Build & Publish Release step fails with Can't copy file .../vendor/.keep because CommandBox's Globber doesn't handle /vendor/** + !/vendor/.keep negation correctly — the parent directory is never created in the temp staging area
  • Replace with /vendor/*/ which ignores subdirectories while keeping top-level files like .keep, avoiding the negation pattern entirely

Test plan

  • CI build step passes without the vendor/.keep copy error

🤖 Generated with Claude Code

CommandBox's Globber fails to copy vendor/.keep when using
/vendor/** with !/vendor/.keep negation — the parent directory
is never created in the temp staging area. Use /vendor/*/ to
ignore subdirectories while keeping top-level files.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Wheels Test Matrix

Engine MySQL PostgreSQL SQL Server H2 CockroachDB Oracle SQLite
lucee6 ❌ 69
lucee7 ❌ 69
adobe2023 --
adobe2025 --
boxlang -- ❌ 78

Results for commit 4918001.

@github-actions
Copy link
Copy Markdown
Contributor

Wheels Test Results

    32 files  ±0   3 360 suites  ±0   14m 52s ⏱️ -30s
66 504 tests ±0  66 007 ✅ ±0  290 💤 ±0  84 ❌ ±0  123 🔥 ±0 
68 424 runs  ±0  67 918 ✅ ±0  290 💤 ±0  93 ❌ ±0  123 🔥 ±0 

For more details on these failures and errors, see this check.

Results for commit b4497be. ± Comparison against base commit ca82e28.

@bpamiri bpamiri merged commit 8586411 into develop Mar 26, 2026
9 of 10 checks passed
@bpamiri bpamiri deleted the peter/fix-build-vendor-keep branch March 26, 2026 14:00
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.

1 participant