Skip to content

fix website build issue#574

Merged
adamavenir merged 1 commit intomainfrom
2.0.5
Mar 24, 2026
Merged

fix website build issue#574
adamavenir merged 1 commit intomainfrom
2.0.5

Conversation

@adamavenir
Copy link
Copy Markdown
Collaborator

…ated having run first

…ated having run first

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 24, 2026 05:15
@adamavenir adamavenir merged commit e433ac2 into main Mar 24, 2026
1 check passed
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the GitHub Pages deployment workflow to ensure the documentation site build has all required project artifacts available during CI.

Changes:

  • Renames the website build step for clarity.
  • Adds root-level dependency install and project build steps to generate required artifacts.
  • Adds an explicit LLM docs build step prior to copying llms*.txt outputs into the deployed site root.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +56 to +58
- name: Build llms docs
run: npm run build:docs

Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

npm run build (root) already runs npm run build:docs as part of the build pipeline (see scripts/build-quiet.js). Running npm run build:docs again here is redundant and increases CI time; consider removing this step, or alternatively replace npm run build with a narrower set of commands if the intent is to only generate LLM docs for Pages.

Suggested change
- name: Build llms docs
run: npm run build:docs

Copilot uses AI. Check for mistakes.
Comment on lines +50 to +52
- name: Install project dependencies
run: npm ci

Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This workflow now runs both npm ci in website/ and npm ci at the repo root, but actions/setup-node caching is keyed only off website/package-lock.json. To avoid unnecessary cache misses and keep caching aligned with both installs, update the node cache key to include the root package-lock.json as well (or remove cache-dependency-path if you want the default behavior).

Copilot uses AI. Check for mistakes.
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.

2 participants