Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cb144e0c49
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "docs:validate": "test -f scalar/index.html", | ||
| "docs:dev": "npm --prefix website run start", | ||
| "docs:build": "npm --prefix website run build", | ||
| "docs:validate": "npm --prefix website run build", |
There was a problem hiding this comment.
Make docs scripts install website dependencies first
On a clean checkout, bun install only installs the root package, so bun run docs:validate now fails with docusaurus: not found because website/node_modules is never installed before running npm --prefix website run build. This regression makes the documented docs workflow unusable unless contributors know to run an extra manual install step (npm --prefix website ci) first.
Useful? React with 👍 / 👎.
|
✅ Docusaurus preview build is ready.
This artifact is generated from the PR branch and can be reviewed before merge. |
Summary
website/.docs/openapi.yamlusingdocusaurus-plugin-openapi-docs.docs:dev,docs:build,docs:validate) to use Docusaurus.website/build.Key Details
npm --prefix website run api:gen.website/docs/apiis gitignored).docs/openapi.yaml(upstream BlueBubbles spec copy).Publishing Behavior
main, not from this PR branch.Validation
bun run checkbun run docs:validatenpm --prefix website run build