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

Code Style #39

Open
astapa opened this issue Aug 20, 2016 · 1 comment
Open

Code Style #39

astapa opened this issue Aug 20, 2016 · 1 comment
Labels
enhancement help wanted Long trem Normal Priority

Comments

@astapa
Copy link
Member

astapa commented Aug 20, 2016

"Sometimes, I can't help but think that I'm not that good. But whenever I look at S.C.S.I's code, it reminds me... that I'm even worse."

The code is a bit of a mess. Or a lot of a mess. Most of it is probably lack of comments. Some of it is undoubtedly comes from the fact that the code is written by 2 people with different ideas about how the code should look. The rest is probably because of said people not taking the time to make their code look exactly as they want it to.

We should probably work on some of that.
@jellopudding
Copy link
Member

We need to discus using PEP 8 for all source code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement help wanted Long trem Normal Priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jellopudding @astapa