Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Update Pylint #334

Open
rhyolight opened this issue Oct 25, 2013 · 0 comments
Open

Update Pylint #334

rhyolight opened this issue Oct 25, 2013 · 0 comments

Comments

@rhyolight
Copy link
Member

Add things like:

  • 3 newlines before top level classes and functions, 2 before methods
  • Check for copyright notice?
  • Add string functions like string.join to the disallowed functions list.
  • spaces after commas
  • Files must end with exactly one newline character.
  • No tabs should be used anywhere.
  • prefer double quotes (not sure exact implementation for that)
  • Wrapping arguments. If some are left on first line then match the indentation of the first one on subsequent lines. If none are left on the first line then add one or two levels of indentation (two levels are required when necessary to differentiate between a new scope on the following line, like arguments in a function definition).
@ghost ghost assigned scottpurdy Oct 25, 2013
@rhyolight rhyolight added P4 and removed build labels Sep 5, 2014
@rhyolight rhyolight added this to the Refactoring milestone Sep 19, 2014
@rhyolight rhyolight modified the milestone: Refactoring Oct 15, 2014
scottpurdy added a commit that referenced this issue Apr 28, 2015
Match pylintrc to Numenta internal, partial resolution for #334
mihail911 pushed a commit to mihail911/nupic that referenced this issue Aug 5, 2015
Match pylintrc to Numenta internal, partial resolution for numenta#334
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants