Skip to content

Conversation

emilk
Copy link
Owner

@emilk emilk commented Oct 15, 2025

Makes it easier to set the same options for many tests

@emilk emilk requested a review from lucasmerlin as a code owner October 15, 2025 08:58
@emilk emilk changed the title Add `egui_kittest::Harness::set_options Add egui_kittest::Harness::set_options Oct 15, 2025
Copy link

github-actions bot commented Oct 15, 2025

Preview available at https://egui-pr-preview.github.io/pr/7638-emilkharness-setoptions
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

View snapshot changes at kitdiff

Copy link
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

Nice! I think this should also be in HarnessBuilder though

@emilk emilk requested a review from lucasmerlin October 15, 2025 09:18
Copy link
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

Nice!

Co-authored-by: Lucas Meurer <[email protected]>
@lucasmerlin lucasmerlin merged commit c79096e into main Oct 15, 2025
47 checks passed
@lucasmerlin lucasmerlin deleted the emilk/harness-set_options branch October 15, 2025 09:42
@emilk emilk changed the title Add egui_kittest::Harness::set_options Add egui_kittest::HarnessBuilder::with_options Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants