Skip to content

Comments

Ch refactor#2

Open
faithngetich wants to merge 52 commits intomasterfrom
ch-refactor
Open

Ch refactor#2
faithngetich wants to merge 52 commits intomasterfrom
ch-refactor

Conversation

@faithngetich
Copy link
Owner

No description provided.

check Outdated
@@ -0,0 +1,10 @@
1. Check if person is in all_people

Choose a reason for hiding this comment

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

What does this file do?
Consider removing it if it does not do anything or if it was for personal reference.

*.py[cod]
*$py.class

# C extensions

Choose a reason for hiding this comment

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

Consider removing files and directories that you do not have in your project.

-d, --database=<sqlite_database> Save state to specified database [default: amity_db]
"""
import cmd, os, sys
from docopt import docopt, DocoptExit

Choose a reason for hiding this comment

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

Consider adding a line of separation between all your import statements, e.g:

# system imports first
import os

#imports from packages imported
from docopt import docopt

#imports from your file system/ structure
from views import Amity

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.

2 participants