search

PyFilesystem / pyfilesystem

Python filesystem abstraction layer
http://pyfilesystem.org/
BSD 3-Clause "New" or "Revised" License
287 stars 63 forks source link

Base class docstrings major update #257

Closed arekbulski closed 8 years ago

arekbulski commented 8 years ago

I reviewed the docstrings in base class again, and fixed all major issues. Please accept it in entirety. We neeed to make documentation better. Ping @lurch

Included closed issue https://github.com/PyFilesystem/pyfilesystem/issues/230 and previous patch https://github.com/PyFilesystem/pyfilesystem/pull/231

lurch commented 8 years ago

(The reason you accidentally added extra commits to this PR, was because you've submitted a PR directly from you master branch, and so whenever you update your master, this PR will get updated too. The normal way of doing things on github to create a new branch (off master), make your changes to the branch, and then PR from there. And then you're free to carry on updating your master branch (as a 'clean copy' of upstream) with any work you do always done on separate branches.)

lurch commented 8 years ago

And I'll try to review this patch one evening this week, if I have time.

arekbulski commented 8 years ago

Reopening under dedicated branch. Thank you for guidance.