Skip to content

Conversation

@jcharlong
Copy link
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.
Simplify if statement condition
Copy link
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?

@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
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