Skip to content

Break CI build link between docs and website#1135

Merged
gantoreno merged 22 commits into
masterfrom
gro-162-break-cibuild-link-between-docs-and-website
May 22, 2026
Merged

Break CI build link between docs and website#1135
gantoreno merged 22 commits into
masterfrom
gro-162-break-cibuild-link-between-docs-and-website

Conversation

@gantoreno
Copy link
Copy Markdown
Contributor

@gantoreno gantoreno commented May 20, 2026

Fixes GRO-162

Up until now, the docs website has been on a half-source-half-output situation, the main website embeds the docs, but docs don't build on their won because it requires shared things from the main website. Initially it appeared that only a handful of files were necessary for this application to properly build on its own. It was more than that. Each layout that got ported over unveiled transitive dependencies that also needed to be ported over.

This is also made worse by the fact that the docs website compiles another bunch of marketing pages that aren't really necessary - in the end, only /docs matters for this case, but the current repo tracks a lot of unrelated stuff.

This PR includes the estimated necessary for the application to build properly, and brings the absolute necessary for the /docs route to build and work. This introduces duplication, but it's the first step towards building independence.

@gantoreno gantoreno requested a review from a team May 20, 2026 01:54
@iethree iethree marked this pull request as draft May 20, 2026 17:41
Copy link
Copy Markdown
Contributor

@iethree iethree left a comment

Choose a reason for hiding this comment

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

Awesome work on a huge task. Just a few things I think need to be addressed.

Comment thread script/fetch-layouts Outdated
Comment thread .github/workflows/update-docs-base.yml
@gantoreno gantoreno force-pushed the gro-162-break-cibuild-link-between-docs-and-website branch from 0c76c75 to a916be3 Compare May 21, 2026 20:18
@gantoreno gantoreno marked this pull request as ready for review May 21, 2026 20:19
@gantoreno gantoreno requested a review from iethree May 21, 2026 20:19
@gantoreno gantoreno requested a review from a team May 21, 2026 20:19
Copy link
Copy Markdown
Contributor

@iethree iethree left a comment

Choose a reason for hiding this comment

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

🚀

@gantoreno gantoreno merged commit b95ad71 into master May 22, 2026
1 of 2 checks passed
@gantoreno gantoreno deleted the gro-162-break-cibuild-link-between-docs-and-website branch May 22, 2026 00:12
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