Skip to content

fix(ci): validate docs before deployment - #171

Merged
ManukMinasyan merged 1 commit into
4.xfrom
fix/docs-dependency-automation
Aug 26, 2026
Merged

fix(ci): validate docs before deployment#171
ManukMinasyan merged 1 commit into
4.xfrom
fix/docs-dependency-automation

Conversation

@ManukMinasyan

Copy link
Copy Markdown
Contributor

Summary

  • restore better-sqlite3 to Docus 5.12.3's supported 12.x range
  • build documentation on every pull request
  • deploy the exact artifact produced by the verified build
  • group only minor and patch Dependabot updates
  • keep major dependency updates separate for manual review
  • serialize build and deployment to prevent stale deployments

Why

PR #170 upgraded better-sqlite3 to 13.0.3, outside Docus's declared peer range.

PHP CI passed because it never installed or built docs/. The deployment later failed during npm ci.

Verification

  • documentation installation and generation passed with Node 24 and npm 11
  • actionlint passed
  • zizmor passed with no findings
  • Dependabot schema validation passed
  • PHP suite passed with 161 tests and 20,685 assertions
  • independent code review returned ready to merge

Follow-up

Require the Docs build check on 4.x, including administrator merges.

Copilot AI lite review requested due to automatic review settings August 26, 2026 17:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ManukMinasyan
ManukMinasyan merged commit 97e7383 into 4.x Aug 26, 2026
11 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.

2 participants