Florian Bruhin

Results 1603 comments of Florian Bruhin

Note this only emits a warning when the exception is *raised* (which would only ever be done in `requests` code), not when it is caught. A module-level `__getattr__` would instead...

This doesn't emit any warnings at all, as shown by a simple test with `python -Werror -c "from requests.exceptions import URLRequired"`. You'll need to rename `URLRequired` to something like `_URLRequired`...

What's the usecase for actually having a list of locations? Doesn't `''` (which is already implemented) suffice?

But do you actually ever need to do that multiple times? If not, `''` does exactly what you describe.

@pkillnine I'm closing this now, I think `''` is sufficient. @m2habert I'd rather just have a search history after typing `/` similar to `:`. There's an issue about it somewhere.

Okay, I'm convinced :wink: - for 1., a completion for `:undo` would probably make more sense though.

Let me note that you can already: - Use `` (`:tab-focus last`) to focus the last focused tab - Use `gt` (`:buffer`) to focus a tab by name - Use...

Can you please elaborate a bit on what exactly was the problem in your case? Was it `build/` as well as outlined in #1978?

> @RonnyPfannschmidt mentions that `PYTEST_CURRENT_TEST` should be reserved for the main thread. That seems to me like it would break too many things. Reviewing [how the env var is used](https://github.com/search?q=PYTEST_CURRENT_TEST&type=code&p=1)...

I wonder if we should introduce a new "recommended" (but discouraged) way to check whether code is running under testing. Either something like a simple `PYTEST=1` env var, or documenting...