Skip to content

fix(docs): trigger deploy with base-path fix#37

Merged
ryanbas21 merged 1 commit into
mainfrom
fix/docs-deploy-trigger
May 12, 2026
Merged

fix(docs): trigger deploy with base-path fix#37
ryanbas21 merged 1 commit into
mainfrom
fix/docs-deploy-trigger

Conversation

@ryanbas21
Copy link
Copy Markdown
Owner

Summary

  • The elm-pages base path patch (baa02c1) fixed route pre-rendering but never deployed — the docs workflow only triggers on apps/docs/** changes, and the fix modified patches/, package.json, and .github/workflows/docs.yml
  • Widens the paths filter to include the workflow file itself and patches/elm-pages*
  • Adds a VS Code Extension link to Getting Started to trigger the deploy

Test plan

  • Merge triggers the docs workflow (verify in Actions tab)
  • https://ryanbas21.github.io/devtools/ loads with proper content (no "Page not found")
  • Subpages like /devtools/docs/getting-started render correctly

🤖 Generated with Claude Code

…th fix

The elm-pages base path patch (baa02c1) fixed route pre-rendering but
never deployed because the docs workflow only triggers on apps/docs/**
changes. The fix modified patches/, package.json, and the workflow file
itself — none of which matched the filter.

- Add .github/workflows/docs.yml and patches/elm-pages* to paths filter
- Add VS Code Extension link to Getting Started page (triggers deploy)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ryanbas21 ryanbas21 merged commit 8e1b372 into main May 12, 2026
1 check passed
@ryanbas21 ryanbas21 deleted the fix/docs-deploy-trigger branch May 12, 2026 20:20
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