Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check style #51

Closed
elg opened this issue Jun 15, 2015 · 1 comment
Closed

Check style #51

elg opened this issue Jun 15, 2015 · 1 comment
Assignees
@VeryTastyTomato
Labels
CODE Minor QUALITY
Milestone
Alpha 0 - install...

Comments

@elg
Copy link
Contributor

elg commented Jun 15, 2015

Do we need to jump a line after docstring on each function/method?
def meth()
"""Doc"""

dostuff()

should maybe become

def meth()
"""Doc"""
dostuff()

for more visibility when we have a lot of methods with only 1 line.
@elg elg added this to the Alpha 0 - installation possible milestone Jun 15, 2015
@VeryTastyTomato
Copy link
Contributor

We don’t need to, in fact in python’s offical doc, examples don’t show an empty line after docstrings.

VeryTastyTomato added a commit to VeryTastyTomato/passhport that referenced this issue Jun 15, 2015
@VeryTastyTomato
Fix issue LibrIT#51
5582ad9
@elg elg closed this as completed Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VeryTastyTomato VeryTastyTomato

Labels
CODE Minor QUALITY
Projects
None yet
Milestone
Alpha 0 - installation possible
Development

No branches or pull requests
2 participants
@elg @VeryTastyTomato