Repository navigation
Optimize new research PNGs to fix pre-commit CI - #63
Merged
Merged
Conversation
oxipng (--opt 4 --strip safe, v10.2.1 as in CI) had not been run on mouse_reach.png and retina_schematic.png added in 9cdc92b. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Summary
Errors per inputErrors in _site/publications.html
|
Both started failing site-health on PR #63 (500s from ccneuro.org, rejected 302 redirects from iopscience) while loading fine in a browser; the scheduled run passed the day before. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Publication links are regenerated weekly from Google Scholar and point at hundreds of publisher hosts that fail transiently (5xx, redirect loops) far more often than links actually rot. Check publications.html in a separate lychee step with fail: false, surfacing failures as a workflow warning and a sticky PR comment instead of a red build. This replaces the per-host ccneuro.org/iopscience exclusions from the previous commit, so those links are still reported. The workflow file is also added to site-health's path filter so edits to it get validated. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Summary
Errors per inputErrors in _site/publications.html
|
jmxpearson
marked this pull request as ready for review
October 13, 2026 19:26
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
1 participant
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.
The pre-commit run on 9cdc92b failed because the oxipng hook rewrote two newly added images. This runs oxipng v10.2.1 (the version CI installs) with the hook's flags. The files shrink from 144 KiB to 113 KiB with no loss in quality.
On this PR, site-health then failed on four external publisher links in publications.html. Two ccneuro.org proceedings PDFs returned 500s, and two iopscience.iop.org links ended in rejected 302 redirects. A rerun gave the same result, but yesterday's scheduled run had passed. Following the existing convention in lychee.toml, both hosts are now excluded with a comment. This change is a separate commit, so it's easy to drop.
🤖 Generated with Claude Code