Skip to content

Add retry option - #56

Merged
mwaskom merged 1 commit into
mainfrom
michael/2026-01-12-add-retries
Jan 26, 2026
Merged

Add retry option#56
mwaskom merged 1 commit into
mainfrom
michael/2026-01-12-add-retries

Conversation

@mwaskom

@mwaskom mwaskom commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

This feature allows you to mark specific tests as flaky so they'll be automatically retried on failure.

@mwaskom
mwaskom requested a review from freider January 12, 2026 16:49
@danielnorberg

Copy link
Copy Markdown

Good Claude!

@mwaskom
mwaskom requested a review from danielnorberg January 12, 2026 16:54
@mwaskom

mwaskom commented Jan 12, 2026

Copy link
Copy Markdown
Contributor Author

It wrote a perfect design and implementation and then got stuck in a death loop trying to test it because it messed up the Python packing :D

@charlesfrye

Copy link
Copy Markdown
Contributor

@mwaskom he just like me fr

@mwaskom

mwaskom commented Jan 16, 2026

Copy link
Copy Markdown
Contributor Author

@prbot approve @freider

@modal-pr-review-automation modal-pr-review-automation Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved 👍. @freider will follow-up review this.

Comment thread README.md
* notest: Exclude the code block from testing.
* fixture:<name>: Apply named pytest fixtures to the code block.
* continuation: Continue from the previous code block, allowing you to carry over state.
* retry:<count>: Automatically retry the test up to the specified number of times if it fails.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

would it make sense to call it flaky to be consistent with @pytest.mark.flaky?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Maybe, although retry:3 feels pretty clear about what's happening while flaky:3 is a bit ambiguous (though not totally impenetrable of course)...

@mwaskom
mwaskom merged commit c9841a0 into main Jan 26, 2026
8 checks passed
@mwaskom
mwaskom deleted the michael/2026-01-12-add-retries branch January 26, 2026 16:41
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.

4 participants