Skip to content

CI churn cleanup - #3103

Merged
RunDevelopment merged 2 commits into
mainfrom
clippy-fixes
Aug 26, 2026
Merged

CI churn cleanup#3103
RunDevelopment merged 2 commits into
mainfrom
clippy-fixes

Conversation

@197g

@197g 197g commented Aug 26, 2026

Copy link
Copy Markdown
Member

Should cleanup CI once more. One clippy fix, and the reference image in Catmul interpolation was no longer accurate after an update to pic-scale-safe very slightly modified a coefficient.

197g added 2 commits August 26, 2026 13:32
An update in pic-scale-safe 0.1.11 slightly modified coefficients.
Comment thread tests/regression.rs
Comment on lines +239 to +246
assert_eq!(resizable, expected, "Output (resized, target): {:?}", {
let path = base.join("output/image/resize-with-alpha-original-half-size.png");
save_image(&path, &resizable).unwrap();
(
path.display().to_string(),
target_path.display().to_string(),
)
});

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks hacky to me but whatever. Not a blocker.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd reuse the abstractions in the regression tests but it's always a bit of a hassle sharing modules between them (and that one has its own harness, so we can't move them over easily either).

@RunDevelopment
RunDevelopment merged commit 0ed9a11 into main Aug 26, 2026
31 checks passed
@RunDevelopment
RunDevelopment deleted the clippy-fixes branch August 26, 2026 11:51
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.

2 participants