-
Notifications
You must be signed in to change notification settings - Fork 31
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
backports: documentation wireframe for release-0.2 branch #72
backports: documentation wireframe for release-0.2 branch #72
Conversation
Copy all documentation related scripts from the main nfd repository. (cherry picked from commit 358a532)
Copy over files from the main nfd repository. (cherry picked from commit 36c3573)
(cherry picked from commit 9a162e5)
Copy over relevant options and targets from the main nfd repo. (cherry picked from commit 1d92d6a)
Modify Jekyll configuration for the operator project. Add stub index page. (cherry picked from commit 07150eb)
Fix site baseurl and redirect to 'stable' version only if it exists. (cherry picked from commit 09c8f1b)
(cherry picked from commit aaa8eb5)
Work around the problematic wrapper scripts inside the jekyll container image that perform all sorts of mischief making us cry. Also, move uid/gid mapping out of SITE_BUILD_CMD in order to enable podman builds, e.g. Add ':Z' to volume mount options to enable build when Selinux is enabled. $ make site-build CONTAINER_RUN_CMD="podman run" (cherry picked from commit 85ab1f9)
Change the sidebar customization so that the full version (e.g. v0.2.0) is displayed, instead of the truncated "releae branch version" (e.g. v0.2) in the top left corner of the page. The items in the version menu are unchanged and will still show the shorter form. Cherry picked from node-feature-discovery repo commit 4d19e1ab85b6cf4c6a2f6e556bf04654e4fb238e. (cherry picked from commit ac98ddc)
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ArangoGutierrez, marquiz The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Necessary cherry-picks from master, without any modifications.
On top of this we can backport #68 to get a minimal (useful) documentation for the latest release (i.e. v0.2.0)