Skip to content

fix(ci): fix docs smoke test and setup comprehensive redirects#38

Merged
ctreffs merged 3 commits intomasterfrom
fix/docs-smoke-test
Feb 11, 2026
Merged

fix(ci): fix docs smoke test and setup comprehensive redirects#38
ctreffs merged 3 commits intomasterfrom
fix/docs-smoke-test

Conversation

@ctreffs
Copy link
Member

@ctreffs ctreffs commented Feb 11, 2026

This PR fixes the documentation smoke test and implements a robust redirect structure for GitHub Pages.

Changes:

  • Fix Smoke Test: Updated the curl check in docs.yml to search for "Documentation" instead of "FirebladeMath". Since the DocC output is a Single Page Application (SPA), the library name is injected via JavaScript and not present in the initial static HTML shell, which caused the previous check to fail.
  • Enhanced Redirects: Implemented a new redirect strategy to support alias and legacy URLs:
    • math/ -> math/latest/documentation/fireblademath/
    • math/latest/ -> math/main/documentation/fireblademath/
    • math/latest/documentation/fireblademath/ -> math/main/documentation/fireblademath/
    • math/documentation/fireblademath/ -> math/main/documentation/fireblademath/
  • Verification: Updated the CI smoke test to verify the new redirect logic at the root URL.

@ctreffs ctreffs changed the title Fix docs-smoke test fix(ci): fix docs smoke test and setup comprehensive redirects Feb 11, 2026
@ctreffs ctreffs marked this pull request as ready for review February 11, 2026 13:03
@ctreffs ctreffs enabled auto-merge (squash) February 11, 2026 13:05
@ctreffs ctreffs disabled auto-merge February 11, 2026 13:21
@ctreffs ctreffs merged commit bf00063 into master Feb 11, 2026
10 checks passed
@ctreffs ctreffs deleted the fix/docs-smoke-test branch February 11, 2026 13:23
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