CLIxIndia-Dev / CLIxIndia-Dev/MiTiRobot

Code style guide ... pep8 standard

Open
#2 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

So this is really nit-picky, but it would help the readability of the code if you followed the [PEP8 style guide](https://www.python.org/dev/peps/pep-0008/). Depending on what editor you use, there should be various plugins that flag pep8 violations in the editor. Here's an example of what it would look like. Hovering over the red line tells me what the editor thinks could be improved.

captura de pantalla 2017-09-15 a la s 3 20 05 pm

I've never actually read the entire pep8 guide, but in general:

* All imports at the top of the file, starting with standard library imports (`import X`) in alphabetical order, then third-party packages, then relative imports.
* Two new lines before any new function / class definition.
* Very picky about spacing and how things should line up.

I can help you find a pep8 plugin and configure it, if you let me know what editor you're using.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.