Skip to content

Run tests in debug mode as well#1146

Merged
bdraco merged 24 commits intomasterfrom
debug_runners_alternative
Apr 10, 2025
Merged

Run tests in debug mode as well#1146
bdraco merged 24 commits intomasterfrom
debug_runners_alternative

Conversation

@bdraco
Copy link
Member

@bdraco bdraco commented Apr 10, 2025

Run tests in debug mode in an additional CI job (alternative)

alternative to #1145 that uses
the existing matrix

bdraco and others added 11 commits April 9, 2025 22:22
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <webknjaz@redhat.com>
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <webknjaz@redhat.com>
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <webknjaz@redhat.com>
alternative to #1145 that uses
the existing matrix
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Apr 10, 2025
@bdraco bdraco changed the title Run tests in debug mode Run tests in debug mode (alternative) Apr 10, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 10, 2025

CodSpeed Performance Report

Merging #1146 will not alter performance

Comparing debug_runners_alternative (aa4a247) with master (cf690d5)

Summary

✅ 244 untouched benchmarks

@bdraco bdraco marked this pull request as ready for review April 10, 2025 10:35
Copy link
Member

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

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

Yeah, I like this variant more than #1145

@bdraco bdraco changed the title Run tests in debug mode (alternative) Run tests in debug mode as well Apr 10, 2025
@bdraco bdraco merged commit 28b7b4b into master Apr 10, 2025
63 checks passed
@bdraco bdraco deleted the debug_runners_alternative branch April 10, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants