Skip to content

Build @ghost/core before docs in Pages deploy#38

Merged
nahiyankhan merged 1 commit intomainfrom
ci/build-ghost-core-before-docs
Apr 20, 2026
Merged

Build @ghost/core before docs in Pages deploy#38
nahiyankhan merged 1 commit intomainfrom
ci/build-ghost-core-before-docs

Conversation

@nahiyankhan
Copy link
Copy Markdown
Collaborator

Summary

  • Pages deploy was failing because apps/docs imports @ghost/core/browser but the workflow only built @ghost/ui first. Without packages/ghost-core/dist/, tsc couldn't resolve the entry and every call into parseFingerprint/lintFingerprint collapsed to any.
  • Adds a Build @ghost/core library step before the @ghost/ui build in .github/workflows/deploy-pages.yml.

Test plan

  • GitHub Pages deploy workflow succeeds on this branch
  • apps/docs builds without TS2307 on @ghost/core/browser

🤖 Generated with Claude Code

The docs site imports @ghost/core/browser for fingerprint parsing,
but the Pages workflow only built @ghost/ui. Without core's dist/,
tsc couldn't resolve the browser entry and every caller of
parseFingerprint/lintFingerprint collapsed to any.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nahiyankhan nahiyankhan merged commit b5c9614 into main Apr 20, 2026
4 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