Skip to content

fix(ci): Vercel ignore step never matched — every deploy was cancelled - #12

Merged
pango07 merged 1 commit into
masterfrom
fix/vercel-ignore-step
Sep 10, 2026
Merged

fix(ci): Vercel ignore step never matched — every deploy was cancelled#12
pango07 merged 1 commit into
masterfrom
fix/vercel-ignore-step

Conversation

@pango07

@pango07 pango07 commented Sep 10, 2026

Copy link
Copy Markdown
Owner

The ignore command diffed `-- landing/`, but Vercel runs it from the project's root directory (`landing`), so it looked for `landing/landing/` and always exited 0 (skip). Every deployment since #2 merged has been CANCELED, including #11 (the landing redesign).

Switched to `git diff HEAD^ HEAD --name-only | grep -q '^landing/'`, which is cwd-independent, and added a copy in `landing/vercel.json`. Merging this touches `landing/`, so it will also trigger the deploy of the redesign.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XSbPaf2fxjrLNjNnn68bXW

The check diffed `-- landing/`, but Vercel runs the ignore command from
the project's root directory (landing), so the path resolved to
landing/landing/ and never matched. Every deployment since the rule
landed was cancelled — including ones that changed the landing page.

Use `--name-only | grep '^landing/'` instead: git prints repo-relative
paths regardless of cwd. Exit 1 (build) when landing/ changed, exit 0
(skip) otherwise. A copy lives in landing/ so it is picked up whether
Vercel reads config from the repo root or the root directory.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XSbPaf2fxjrLNjNnn68bXW
@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
flicky Ready Ready Preview Sep 10, 2026 9:16pm UTC

Request Review

@pango07
pango07 merged commit 049365c into master Sep 10, 2026
4 of 5 checks passed
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