Apply ruff/flake8-bugbear preview rule B901

B901 Using `yield` and `return` in a generator function can lead to confusing behavior

Co-authored-by: Martin Durant <[email protected]>

Author: DimitriPapadopoulos

fsspec/spec.py

@@ -428,11 +428,9 @@ def walk(self, path, maxdepth=None, topdown=True, on_error="omit", **kwargs):
         except (FileNotFoundError, OSError) as e:
             if on_error == "raise":
                 raise
-            elif callable(on_error):
+            if callable(on_error):
                 on_error(e)
-            if detail:
-                return path, {}, {}
-            return path, [], []
+            return (path, {}, {}) if detail else (path, [], [])  # noqa: B901
 
         for info in listing:
             # each info name must be at least [path]/part , but here