Skip to content

fix(docs): configure Astro site/base for GitHub Pages#30

Merged
kattsushi merged 1 commit into
masterfrom
fix/astro-pages-base
Feb 7, 2026
Merged

fix(docs): configure Astro site/base for GitHub Pages#30
kattsushi merged 1 commit into
masterfrom
fix/astro-pages-base

Conversation

@kattsushi

@kattsushi kattsushi commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

Configure base=/effectify/ and site=https://devx-op.github.io/effectify/ in apps/docs/astro.config.ts so Astro generates asset URLs within the project subpath. This fixes blocked modules due to HTML 404 responses (MIME type text/html) and ensures JS/CSS load on GitHub Pages.

Summary by CodeRabbit

  • Chores
    • Updated documentation site configuration settings.

@coderabbitai

coderabbitai Bot commented Feb 7, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Adds site and base configuration properties to the Astro docs configuration, setting the site URL to the GitHub Pages deployment location and the base path to "/effectify/". These properties configure deployment settings for the documentation site.

Changes

Cohort / File(s) Summary
Astro Configuration
apps/docs/astro.config.ts
Added site property pointing to the GitHub Pages URL and base property set to the deployment base path "/effectify/".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Hoppy configuration hops,
Site and base now set with care,
GitHub Pages awaits with glee,
Effectify takes flight through the air!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'fix(docs): configure Astro site/base for GitHub Pages' accurately and specifically describes the main change: configuring Astro's site and base properties for GitHub Pages deployment.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/astro-pages-base

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud

nx-cloud Bot commented Feb 7, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit b010cf8

Command Status Duration Result
nx affected --target=test --base=87c40a1ce45893... ✅ Succeeded 10s View ↗
nx affected --target=build --base=87c40a1ce4589... ✅ Succeeded 17s View ↗
nx affected --target=typecheck --base=origin/ma... ✅ Succeeded <1s View ↗
nx affected --target=lint --base=origin/master ... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-07 15:51:30 UTC

@kattsushi kattsushi merged commit f34c59f into master Feb 7, 2026
7 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