Skip to content
This repository was archived by the owner on Mar 31, 2020. It is now read-only.

RED REINDEER #1

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

RED REINDEER #1

wants to merge 31 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 21, 2019

No description provided.

@ghost ghost changed the title WIP: RED REINDEER RED REINDEER Mar 4, 2019
@jb3
Copy link
Member

jb3 commented Mar 7, 2019

Interesting project idea! You have used some documentation though more would be a nice thing to have.

Your functions do not follow the naming convention of function_name, most are like functionName.

You recommend running the project by directly executing project/__main__.py though you should accomplish the same thing using the recommended python -m project.

Also, I can't really get my head around why there is a rather large postscript document in your project, was this left in intentionally? Is it used somewhere?

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.

1 participant