Skip to content

None docstrings fix#414

Merged
althonos merged 7 commits into
PyFilesystem:masterfrom
jcharlong:none-docstrings-fix
Sep 25, 2020
Merged

None docstrings fix#414
althonos merged 7 commits into
PyFilesystem:masterfrom
jcharlong:none-docstrings-fix

Conversation

@jcharlong
Copy link
Copy Markdown
Contributor

@jcharlong jcharlong commented Jul 30, 2020

Type of changes

  • Bug fix
  • New feature
  • Documentation / docstrings
  • Tests
  • Other

Checklist

  • I've run the latest black with default args on new code.
  • I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • I've added tests for new code.
  • I accept that @willmcgugan may be pedantic in the code review.

Description

Fixes an issue when docstrings are set to None. Issue #415.

Support "__doc__" being set to `None`, as is done when `-OO` is supplied to the CPython interpreter.
Comment thread fs/base.py Outdated
Simplify if statement condition
Copy link
Copy Markdown
Member

@althonos althonos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job, thanks! Just waiting for you to revert changes on the fs._version file and then I'll merge. Can you add yourself to the CONTRIBUTORS.md file as well if possible?

Comment thread fs/_version.py Outdated
@althonos althonos added this to the v2.4.12 milestone Sep 17, 2020
Revert version bump since it is done as part of the merge/release process.
Add self & maintain alphabetical order.
@jcharlong
Copy link
Copy Markdown
Contributor Author

Reverted the version bump & added myself to the contributors. 👍

@althonos althonos merged commit 34084cf into PyFilesystem:master Sep 25, 2020
@jcharlong jcharlong deleted the none-docstrings-fix branch June 26, 2021 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants