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

Linting organisation and fixes #1065

Merged
merged 2 commits into from
Apr 13, 2020
Merged

Linting organisation and fixes #1065

merged 2 commits into from
Apr 13, 2020

Conversation

AndreMiras
Copy link
Member

  • Organises linter errors and check tests/ directory
  • PEP8 fixes
    • E129 visually indented line with same indent as next logical line
    • E226 missing whitespace around arithmetic operator
    • E241 multiple spaces after ','
    • E262 inline comment should start with '# '
    • E301 expected 1 blank line, found 0
    • E303 too many blank lines
    • E502 the backslash is redundant between bracket
    • E741 ambiguous variable name
    • W391 blank line at end of file

@AndreMiras
Organises linter errors and check tests/ directory
ef63f13
@AndreMiras
PEP8 fixes
6b03c54 
- E129 visually indented line with same indent as next logical line
- E226 missing whitespace around arithmetic operator
- E241 multiple spaces after ','
- E262 inline comment should start with '# '
- E301 expected 1 blank line, found 0
- E303 too many blank lines
- E502 the backslash is redundant between bracket
- E741 ambiguous variable name
- W391 blank line at end of file
@AndreMiras AndreMiras merged commit 58774d8 into kivy:master Apr 13, 2020
@AndreMiras AndreMiras deleted the feature/linting_organisation_and_fixes branch April 13, 2020 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inclement inclement inclement approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AndreMiras @inclement