DaveBoy

> Can you please put your changes into a PR? I'm very happy to do this, I'll sort out the relevant content and submit it to pr tomorrow

> Ah, OK. Probably in fsspec.implementations.github , the url passed to requests in _open, ls and cat should be url-encoded . Would you like to try this? get is ok,...

``` def cat(self, path, recursive=False, on_error="raise", **kwargs): paths = self.expand_path(path, recursive=recursive) urls = [ self.rurl.format(org=self.org, repo=self.repo, path=u, sha=self.root) for u, sh in paths ] ``` @martindurant Here "ValueError: too many...

> `for u, sh in paths` seems simply wrong, it should be `for u in paths` as you suspected. `expand_path` returns "list of all matching paths". @martindurant ``` for u,...

by lazy 加上同步关键字sy