Skip to content

tox update and linter fixes #1674

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.

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

Merged

Conversation

AndreMiras
Copy link
Member

@AndreMiras AndreMiras commented Feb 4, 2019

  • F841 local variable ... is assigned to but never used
  • F812 list comprehension redefines ...

Also added pyleveldb recipe to list of broken recipes, the error was:

/home/user/.local/share/python-for-android/build/other_builds/hostpython2/desktop/hostpython2/Include/Python.h:47:19:
fatal error: crypt.h: No such file or directory

See https://api.travis-ci.org/v3/job/488398065/log.txt

@AndreMiras AndreMiras requested a review from inclement February 4, 2019 08:53
- F841 local variable ... is assigned to but never used
- F812 list comprehension redefines ...

Also added pyleveldb recipe to list of broken recipes, the error was:
```
/home/user/.local/share/python-for-android/build/other_builds/hostpython2/desktop/hostpython2/Include/Python.h:47:19:
fatal error: crypt.h: No such file or directory
```
See https://api.travis-ci.org/v3/job/488398065/log.txt
@AndreMiras AndreMiras force-pushed the feature/tox_update_and_linter_fixes branch from 1014eb9 to b54a1d0 Compare February 4, 2019 09:50
@AndreMiras
Copy link
Member Author

Build eventually green https://travis-ci.org/kivy/python-for-android/builds/488417097 @inclement could you review/merge?

Copy link
Contributor

@KeyWeeUsr KeyWeeUsr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check via grep or something that the parser variables (add_parser() result) are not imported somewhere else, otherwise it seems ok. 👍

@AndreMiras
Copy link
Member Author

Thanks for the review guys!
Yes @KeyWeeUsr I searched the add_parser() result via vim * (the highlighter) and it seamed safe. In the worst case we'll earn reputation points on stackoverflow 😂

@AndreMiras AndreMiras merged commit 0fef037 into kivy:master Feb 4, 2019
@AndreMiras AndreMiras deleted the feature/tox_update_and_linter_fixes branch February 4, 2019 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants