Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Allow docstring only classes. Fixes issue #8. #23

Merged
merged 2 commits into from
Jan 11, 2013
Merged

Allow docstring only classes. Fixes issue #8. #23

merged 2 commits into from
Jan 11, 2013

Conversation

bboe
Copy link
Contributor

@bboe bboe commented Jan 7, 2013

Behaves as expected against all of the cases in the linked file:
https://gist.github.com/4472121

Fixes issue #8.

@JensRantil
Copy link
Contributor

@halst Maybe we would like a test case for this?

@keleshev
Copy link
Contributor

@bboe if you provide automatic tests, I'll merge this. If you don't, I will still merge it, though :-)

@bboe
Copy link
Contributor Author

bboe commented Jan 11, 2013

@halst Tests added.

keleshev pushed a commit that referenced this pull request Jan 11, 2013
Allow docstring only classes. Fixes issue #8.
@keleshev keleshev merged commit 62f948f into PyCQA:master Jan 11, 2013
@keleshev
Copy link
Contributor

@bboe thanks for the patch! Are you using pep257 as a part of CI?

@bboe
Copy link
Contributor Author

bboe commented Jan 11, 2013

@halst At the moment no, it's just strongly encouraged that all pull requests for PRAW pass pep257 now, as well as flake8 and pylint tests. I suppose I could incorporate it into the travis build to automatically test it, but I haven't yet (due to this bug).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants