Skip to content

Example PR for testing kitdiff (do not merge) - #35

Draft
lucasmerlin wants to merge 15 commits into
lucas/github-file-loaderfrom
lucas/example-pr
Draft

lucasmerlin wants to merge 15 commits into
lucas/github-file-loaderfrom
lucas/example-pr

Conversation

@lucasmerlin

@lucasmerlin lucasmerlin commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

This PR exists to test kitdiff's PR features. Do not merge it.

Open it in kitdiff: https://rerun-io.github.io/kitdiff/?url=https://github.com/rerun-io/kitdiff/pull/35

It changes the snapshots of the kittest tests in tests/snapshots in every way we could think of. Each case has its own commit:

Commit File What kitdiff should show
Change a snapshot home.png A red box in the diff
Change a snapshot below the diff threshold viewer_blend_all.png "All differences below threshold!" (5 pixels, each off by 1)
Change the size of a snapshot viewer_old.png "Diff failed: Images must have the same dimensions" (1000×600 → 1000×560)
Change a whole snapshot viewer_click_in_file_tree.png Every pixel differs (inverted colors)
Change only the alpha of a snapshot viewer_filter.png Same colors, 50% alpha
Remove a snapshot viewer_new.png Only an old image
Rename a snapshot viewer_diff.png → viewer_diff_view.png The same image on both sides
Move and change a snapshot viewer_next_snapshot.png → selection/viewer_next.png A removed and an added file: git can't detect renames of changed git-lfs files, as the pointer changes completely
Change only the file mode of a snapshot viewer_pixel_size.png (chmod +x) No difference
Add snapshots with special paths and formats added/new snapshot äöü #1.png, added/deeply/nested/folder/nested.png, added/grayscale_16bit.png Percent-encoded paths, a deep folder, a 16-bit grayscale PNG
Change a file that is not in git-lfs tests/fixtures/widgets/button.new.png Loads from raw.githubusercontent.com, not media.githubusercontent.com
Add a snapshot larger than 1 MB large/noise.png (~3 MB) Loads. The contents API handles files over 1 MB differently
Add a broken PNG added/broken.png (text) An error for this file only
Add files that are not PNGs notes.md, photo.jpg Nothing: kitdiff only lists .png files
Add 120 snapshots bulk/bulk_000.png … bulk_119.png More than one page (100) of PR files. Good for testing how fast you can skip through the snapshots

viewer_magnifier.png does not change, so kitdiff must not list it.

The snapshot tests fail on this PR, so CI uploads a test-results artifact. That tests opening artifacts from the commit list. The "Commit the updated snapshots" button will fail: update_kittest_snapshots.yml does not exist in this repository yet.

The base branch is lucas/github-file-loader, which adds the snapshot tests.

🤖 Generated with Claude Code

@lucasmerlin
lucasmerlin force-pushed the lucas/example-pr branch 2 times, most recently from 410b7c1 to 0064309 Compare October 9, 2026 19:23
lucasmerlin and others added 15 commits October 9, 2026 21:44
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant