geriom
commented
2 years ago I fixed the problems we had in #303
- The build was failing because of the outdated Hugo version. I bumped the Hugo version from 0.53 to 89.4.
- The tab widget we had in tekton.dev was clashing with the latest tab widget included in Docsy. The one in Docsy is less flexible and only allows code. I had to "hack" a solution and create a new tab widget that uses the same styles. This way we don't have to make any updates in the docs, it just works.
- The "Edit this page" broken links are fixed for pages within the website repo. I added a config line to
config.tomlto update the default branch from "master" to "main". This combined with the change @afrittoli suggested in this comment should fix most of the 404s.
tekton-robot
afrittoli
This is the same as #303. Because of the size of this PR, we are going to test it in this branch first before we merge it to main.
The docsy theme was last updated 2 years ago, a few quality of life improvements have been introduced since then, for example:
Changes
This commit pulls the latest version of the theme from the docsy repo with a few changes to facilitate further updates.
Docsy uses Bootstrap and FontAwesome as git submodules, these are checked in. To decouple the dependencies, I removed the submodules and made them npm dependecies. See the changes in
package.jsonandthemes/docsy/assets/scss/main.scss.I also made some changes in
layouts/docs/baseof.htmlfor the site to be compatible with the latest version of the theme.Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
See the contribution guide for more details.