-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: preview in PRs #96
base: main
Are you sure you want to change the base?
Conversation
gh pages generation is set to workflow currently: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the fast fix. Really good job done :)
I guess I'll change this PR to try uploading both ways. |
d038726
to
21b92d0
Compare
Current State
Documentation is built during PRs, and currently, an artifact is uploaded, with a link to the artifact added to the PR. However, this generates numerous links (e.g., see #94), and to view the documentation, users must download and extract the artifact manually.
Desired State
Provide a direct link to the documentation preview in PRs. This link should allow users to preview the documentation directly, without extra steps, and remain stable throughout the PR's lifetime, unaffected by new commits.
Solution
Since GitHub Pages supports only a single folder or artifact per repository, and rebuilding all versions of documentation for every PR is inefficient, a new approach is needed. The gh-pages branch will act as persistent storage for documentation artifacts. Each PR will store its specific documentation in a dedicated subfolder within this branch.
When a PR is updated, the workflow will add or update the corresponding documentation in its designated subfolder. Once the PR is closed or merged, its documentation subfolder will be removed to keep the branch manageable. This approach ensures stable links, efficient builds, and clear documentation previews.
This probably belongs to #78. And it closes #94.