Skip to content

Fix handling of 'forever' delay (fixes #125) - #289

Open
johbo wants to merge 1 commit into
lepture:masterfrom
johbo:delay-forever
Open

johbo wants to merge 1 commit into
lepture:masterfrom
johbo:delay-forever

Conversation

@johbo

@johbo johbo commented Sep 20, 2026

Copy link
Copy Markdown

The sentinel was filtered out by the isinstance check that collects numeric delays, so it never reached poll_tasks and a watch asking not to reload the browser still reloaded it.

Based on #253, keeping the sentinel out of the numeric set so that max() never sees a string.

Comment thread tests/test_watcher.py Outdated
The sentinel was filtered out by the isinstance check that collects
numeric delays, so it never reached poll_tasks and a watch asking not
to reload the browser still reloaded it.

Based on lepture#253, keeping the sentinel out of the numeric set so that
max() never sees a string.

This branch has not been deployed

No deployments
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