Repository navigation
ci(docs): check production and preview builds on pull requests - #777
Open
myasnikovdaniil wants to merge 2 commits into
Open
myasnikovdaniil wants to merge 2 commits into
myasnikovdaniil wants to merge 2 commits into
Conversation
Build the preview tree on pull requests so broken refs under next/ are caught before merge, and check the production output that Pages publishes. Keep build jobs read-only and gate the Pages artifact and deployment to main. Fail dependency installation instead of continuing after npm errors. Assisted-By: LLM Signed-off-by: Myasnikov Daniil <myasnikovdaniil2001@gmail.com>
myasnikovdaniil
requested review from
Andrei Kvapil (kvaps),
Timofei Larkin (lllamnyp) and
Timur Tukaev (tym83)
as code owners
October 26, 2026 09:47
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the
 |
myasnikovdaniil
marked this pull request as draft
October 26, 2026 11:53
The production matrix environment leaked into fixture subprocesses and made preview cases render production metadata. Keep the mode variables on the full-site build step so each fixture can choose its own mode. Assisted-By: LLM Signed-off-by: Myasnikov Daniil <myasnikovdaniil2001@gmail.com>
myasnikovdaniil
marked this pull request as ready for review
October 26, 2026 12:19
1 of 21 tasks
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PRs currently never run the Pages build, and production skips
docs/next/. I added production and preview checks to the existing workflow, including legacy URLs and version rollover.PR jobs use read-only permissions, publication stays on
mainafter both builds pass.Verified locally with Hugo extended 0.164.0, 26 fixtures, five output checks per environment, 129 translator tests and actionlint. Broken references in
next/fail preview, and failed npm installs now stop the build.Stacked on #776, addresses #496 and #653 as part of #773.