Repository navigation
chore: modernize site and fix issues - #482
Merged
Merged
Conversation
- Rebase head.html on just-the-docs 0.12 (fixes duplicate <title>, adds nav activation styles) - Fix web manifest icon paths, FAQ link, and http links - Remove default Jekyll post, jekyll-feed, and obsolete icon files - Add `rake check` for internal links and run it in PR CI - Bump Ruby to 3.4 in CI and Read the Docs - Update README Assisted-by: ClaudeCode:claude-opus-5-5
eduardo-rodrigues
approved these changes
Oct 24, 2026
eduardo-rodrigues
left a comment
eduardo-rodrigues
left a comment
Member
There was a problem hiding this comment.
Better and simpler is always better :-). You're the expert here, I 100% trust you.
It all looks good from a little browse of the site preview.
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
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.
🤖 AI text below 🤖
Cleanup and fixes found while checking the site.
Fixes
_includes/head.htmlwas an old copy of the theme file. It printed a second<title>on every page and did not load the 0.12 nav styles (head-nav stylesheet and nav activation). It is now based on the just-the-docs 0.12 file and keeps the light/dark scheme switch. The JS fallback for browsers withoutprefers-color-schemeis removed./instead of/assets/, and the name was empty.http://links changed tohttps://; CHEP 2019 now points to Indico, because its site is http only.steps.pages.outputs.base_pathwithout apagesstep.Cleanup
jekyll-feed(the feed would be empty).browserconfig.xml,mstile-150x150.png,safari-pinned-tab.svg, a duplicateassets/favicon.ico, the unused Google Analytics block, and theX-UA-Compatibletag.README.md,RakefileandLICENSEare no longer copied into_site.Gemfileand_config.yml.Updates
rake checktask (internal links only), now run in PR CI.rake teststill checks external links.ubuntu-24.04.I checked the local build with
jekyll serve: every linked asset and page returns 200. I did not do a visual check in a browser.