Skip to content

Fix documentation of Control.Concurrent.Async.Warden.shutdown#183

Open
sol wants to merge 1 commit into
simonmar:masterfrom
sol:warden-docs
Open

Fix documentation of Control.Concurrent.Async.Warden.shutdown#183
sol wants to merge 1 commit into
simonmar:masterfrom
sol:warden-docs

Conversation

@sol
Copy link
Copy Markdown

@sol sol commented May 19, 2026

I changed the documentation to what the current code does.

However, personally, I think I would prefer the documented behavior:

  • Provided you use withWarden, misusing this takes such deliberate action that I'm not convinced it's worth protecting against it.
  • Going with the documented behavior:
    • keeps the documentation (and by extension behavior) simpler
    • allows you to remove WardenException entirely

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.

1 participant