Philip Couling

Its a pity this is out of scope since from a user perspective it's less obvious why this should be the case. I'm not arguing against the developer logic though,...

In another world, other apps than wordpress (including nextcloud) have got round this by flagging individual users as being provided by LDAP so that non-ldap users aren't affected. If it...

@bilbo-the-hobbit Yes I agree *hope* is bad currency in software. Standards on the other hand... https://tools.ietf.org/html/rfc3112

I actually looked into doing just that and discovered Wordpres's own code makes it impossible because one of the pluggable functions is circumvented where it shouldn't be. I opened a...

I don't think you need an awaitable `os.read()`. The key rule with asyncio is not to block unless you `await`. But the way you are reading doesn't block. The code...

This has been open quite a long time now. I realise v4.0 refactor may still be ongoing. Earlier in this thread @agronholm mentioned pitfalls of around DST unexpected behaviour. There's...

For those of us not familiar with the project, is there a comment on why this issue was closed besides being open for a long time with little progress? Has...

Indeed I wonder if it's really right to report `pass` lines as no coverage at all. Certainly reporting an abstract method with a single `pass` line is overkill. It would...

For future readers, I've just realised this can easily by done by adding `pass` to `exclude_lines` in `.coveragerc` (described [here](https://coverage.readthedocs.io/en/latest/config.html) and [here](https://coverage.readthedocs.io/en/coverage-5.2.1/excluding.html)): ``` [report] exclude_lines = # Skip any pass...

Another approach, which I now prefer, is to remove the `pass` altogether. It's not required if the method has a doc string. Future readers may prefer to simply document their...