fohrloop/wakepy

Include `on_fail="warn"` in the examples of the README / docs?

Closed this issue · 1 comments

Not all people read all the documentation. Adding on_fail="warn" in the examples, like this:

from wakepy import keep

with keep.running(on_fail="warn"):
    # Do something that takes a long time. The system may start screensaver
    # / screenlock or blank the screen, but CPU will keep running.

would make people notice the on_fail action flag. That's also a good default for applications that are used on multiple platforms, and would probably prevent unnecessary issues and save everyone's time.

I think this is not so acute anymore after the default on_fail action was changed to "warn" (in #388). Perhaps better to keep the examples minimalistic. There parameter is documented already in the User Guide and in the API Reference.