Improve fsspec.open Code Snippet for Better Compatibility

[danyeaw]

Closes #115.

Changes

Updates the Insert open Code Snippet to:

• Use binary mode ("rb") instead of text mode ("rt")
• Replace iteration with ... (pass) to give a place for you to write your code without an error
• Fix typos with the response body for some of the neighboring tests

Before:

with fsspec.open("file:///My/file/path/CHANGE_LOG.md", "rt") as f:
   for line in f:
      print(line)

After:

with fsspec.open("file:///My/file/path/CHANGE_LOG.md", "rb") as f:
   ...

Rationale

• Improved compatibility**: Binary mode works with both text and binary files
• Prevents errors: For-loop iteration over non-text files (like parquet) would fail

[martindurant]

Perhaps instead of print(f.readline()) have simply ... ?

Note that the snippet is not actually useful without kwargs in many cases (e.g., even anon=True for s3). Passing the kwargs to the client was suggested first in #113 , and now included in #116 as soon as @RRosio pushes her work.

[danyeaw]

Hi @martindurant, good idea, I updated the PR with ....

Nice, those sound like some nice updates on the way!