Skip to content

Simple timeout combinator.#918

Open
fisx wants to merge 1 commit into
hspec:mainfrom
wireapp:timeout
Open

Simple timeout combinator.#918
fisx wants to merge 1 commit into
hspec:mainfrom
wireapp:timeout

Conversation

@fisx
Copy link
Copy Markdown

@fisx fisx commented Apr 11, 2025

Fixes #914

@fisx
Copy link
Copy Markdown
Author

fisx commented May 29, 2025

Not sure this is all according to your taste, but having this would have helped me personally to stay out of your hair. :)

@parsonsmatt
Copy link
Copy Markdown

I think Hooks is the wrong location for this. contrib package may be a better fit

@sol
Copy link
Copy Markdown
Member

sol commented Sep 19, 2025

@parsonsmatt I think we just need one extension that adds --timeout=<seconds> and I guess some timeout combinator Timeout.apply :: Seconds -> SpeWith a -> SpeWith a (or maybe Timeout.seconds :: Seconds -> SpeWith a -> SpeWith a, not sure).

Either as a separate package or in -contrib.

I think that will supersum #889, #914, #918 and #927.

At some point @BebeSparkelSparkel wanted to look into it #890 (comment), but from what I understand it's still up for grabs.

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.

timeouts for tests

3 participants