Skip to content

ci(ecosystem-urls): install deps before running check script#59

Merged
rickstaa merged 1 commit intomainfrom
fix/ecosystem-urls-workflow-deps
Apr 28, 2026
Merged

ci(ecosystem-urls): install deps before running check script#59
rickstaa merged 1 commit intomainfrom
fix/ecosystem-urls-workflow-deps

Conversation

@rickstaa
Copy link
Copy Markdown
Member

@rickstaa rickstaa commented Apr 28, 2026

Summary

  • The Check Ecosystem URLs workflow runs node scripts/check-ecosystem-urls.mjs on a fresh runner without installing dependencies.
  • Since commit 7228456 refactored the script to import gray-matter, every workflow run has failed with ERR_MODULE_NOT_FOUND (visible on PR feat(ecosystem): add Flipsuite #55).
  • Adds actions/setup-node@v4 + npm ci so the script can resolve its imports.

Test plan

  • Workflow run triggered by this PR completes successfully
  • Re-run on main after merge still passes (weekly cron)

🤖 Generated with Claude Code

The check-ecosystem-urls workflow ran `node scripts/check-ecosystem-urls.mjs`
on a fresh runner without installing dependencies. Since the script was
refactored to use `gray-matter` (commit 7228456), the workflow has failed
on every trigger with `ERR_MODULE_NOT_FOUND`.

Add `actions/setup-node@v4` and `npm ci` steps so the script can resolve
its imports.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rickstaa rickstaa requested a review from adamsoffer as a code owner April 28, 2026 22:12
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 28, 2026

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

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Apr 28, 2026 10:13pm

Request Review

@rickstaa rickstaa merged commit 912bc6f into main Apr 28, 2026
2 checks passed
@rickstaa rickstaa deleted the fix/ecosystem-urls-workflow-deps branch April 28, 2026 22:13
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