chore(docs): refresh blog visual baseline after content change#3257
Merged
wheels-bot[bot] merged 1 commit intoJun 24, 2026
Merged
Conversation
Contributor
Author
There was a problem hiding this comment.
Wheels Bot — Reviewer
TL;DR — This PR refreshes the committed visual-regression baseline at web/tests/visual-baselines/blog.png after a blog-content change (commit f902f249c, the "Pagination That Isn't a Pain" post), so subsequent web/** PRs stop failing the Visual regression check on a stale diff (issue #2915). It's an auto-generated, single-binary-file change opened by web-deploy.yml. Verdict: comment — nothing blocking; the change is structurally sound and matches its stated purpose.
Correctness
- The diff is a single binary swap of
web/tests/visual-baselines/blog.png. I verified the new blob is a valid PNG of the expected dimensions (PNG image data, 1280 x 800, 8-bit/color RGB, non-interlaced, 120,615 bytes vs. the prior 128,776). The replacement is structurally well-formed and matches the baseline format the visual suite expects. - One inherent limitation worth stating honestly: a static review can confirm the file is a valid PNG, but it cannot confirm the pixels actually correspond to the current blog index render. That verification is exactly what the Visual regression check does on the next
web/**PR — so the baseline is effectively self-validating going forward. No action needed; flagging only so the "comment" verdict (vs. approve) is understood.
Commits
chore(docs): auto-refresh blog visual baseline after content changeconforms to commitlint (typechore, optional scopedocs, sentence-case subject, header well under 100 chars). The PR title (chore(docs): refresh blog visual baseline after content change) is likewise a valid conventional-commit header — relevant since the repo squash-merges and the title becomes the landing subject.- Minor: the commit carries no
Signed-off-by:trailer. DCO sign-off is project policy (not a gated CI check), and this is an automation-authored PR, so this is informational rather than fix-before-merge — but worth noting if theweb-deploy.ymlcommit step can be made tocommit -s.
No tests, docs, changelog, or cross-engine concerns apply: this is test-fixture (baseline) data, not shippable code, and a chore rather than a user-facing feat/fix.
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.
Summary
A blog-content merge changed the blog index page, so the committed visual-regression baseline at
web/tests/visual-baselines/blog.pngis stale. Without this refresh, every subsequentweb/**-touching PR fails the Visual regression check on a deterministic diff until someone refreshes manually (issue #2915).develop's branch protection rejects direct pushes, so the refresh lands through this PR, opened and merged by the deploy run that published the blog change.
🤖 Opened automatically by .github/workflows/web-deploy.yml.