Style guide?

[delirious-lettuce]

This looks like a neat project but I am curious about the style of the code itself and since the project only seems to be four days old, this might be a good time to make changes.

For example:

kaggle-api/kaggle/api/kaggle_api_extended.py

Line 54 in 4625bc7

def competitionsListCli(self, page = 1, search = '', csv = False):

• Python projects usually use four space indentation but this method alone has three different indentations (4 space then 2 space then 3 space).

  Use 4 spaces per indentation level.

• There are also spaces around the = operators whereas PEP8 recommends no spaces.

  Don’t use spaces around the = sign when used to indicate a keyword argument or a default parameter value.

• The function name also uses camelCase instead of the more common snake_case convention.

  Function names should be lowercase, with words separated by underscores as necessary to improve readability.

Many projects seem to stay pretty close to PEP8 except maybe loosening the line length recommendation from 79 chars to around 100 chars.

• http://pep8.org/#maximum-line-length

These are small things but fixing them would touch many lines of code in the project and I wasn't sure if that would be frowned upon. Any thoughts?

[pulkitmaloo]

Can I work on making the code PEP8 Style?

[ghost]

Apologies for the late response. I'm picking up this project now.

Conforming the handwritten code to Google's python style is on my short-term todo list. At least one version will not contain breaking changes, but I don't want to leave in the old method names too long.

[ghost]

Fixed in 4e29c89. Wound up just including the breaking changes. Please see the changelog - they shouldn't affect command line only use.

The code was formatted per Google style, which has a lot of similarities to pep8, but is not identical.