Preview the website on documentation pull requests - #8070
Open
dimitrieh wants to merge 10 commits into
Open
Conversation
test-with-website already checked out FlowFuse/website next to this repository and built the whole site with the pull request's docs, because docs-sync.mjs prefers a sibling checkout over cloning main. It then threw the output away after the link check. That build is now FlowFuse/website's site-preview.yml, called from here, so this repository stops carrying a second copy of the website pipeline and picks up changes to it as the Nuxt migration progresses. When the caller is a pull request from a branch in this repository it also uploads the result to Netlify under a docs-pr-<number> alias and reports the URL on the pull request. The upload is an aliased, unpublished deploy: no --prod, and no --build, so netlify.toml's build command (which reindexes Algolia) never runs against a preview and flowfuse.com keeps serving whatever it served before. test-with-website survives as a gate job so the check name and the publish dependency are unchanged.
Without this the pull request that changes docs.yml does not trigger it, so the build and the preview upload go unverified.
5 tasks
Contributor
Author
Contributor
Author
site-preview.yml no longer exists on FlowFuse/website. The definition moved to FlowFuse/github-actions-workflows as build_website.yml, so the old reference resolved to nothing. The inputs and secrets are unchanged, only the ref moves.
ppawlowski
requested changes
Aug 14, 2026
Contributor
Author
The gate existed only to keep publishing the old check name. A reusable workflow reports as "caller job / called job", so the website build now reports as "Build the website with these docs / Build and check" and publish depends on it directly.
Member
|
@dimitrieh Seems convoluted, shouldn't we just move the docs to the website repo? |
Contributor
Author
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
3 participants
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.
Contributes to Website preview on documentation pull requests.
Replaces the copy of the website build in
docs.ymlwith a call tobuild_website/v1in FlowFuse/github-actions-workflows, and uploads the result to Netlify under adocs-pr-<number>alias so documentation pull requests get a preview URL.The deploy is aliased and unpublished, so netlify.toml's build command never runs against it and flowfuse.com keeps serving what it served before. Fork pull requests receive no secrets, so they build and link-check without uploading.
Draft until:
NETLIFY_AUTH_TOKENandNETLIFY_SITE_IDare available to this repositoryBuild the website with these docs / Build and checkin place ofTest Documentation with website