Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@AndreMiras
Copy link
Member

The recent flake8==3.8.0 update reported new errors:

pythonforandroid/graph.py:133:27: E741 ambiguous variable name 'l'
pythonforandroid/toolchain.py:219:25: E741 ambiguous variable name 'l'
pythonforandroid/toolchain.py:1146:19: F522 '...'.format(...) has unused named argument(s): Fore
pythonforandroid/pythonpackage.py:690:23: F523 '...'.format(...) has unused arguments at position(s): 0
pythonforandroid/logger.py:202:36: E741 ambiguous variable name 'l'
pythonforandroid/logger.py:204:36: E741 ambiguous variable name 'l'
pythonforandroid/archs.py:150:25: E741 ambiguous variable name 'l'
pythonforandroid/recipes/android/src/android/_ctypes_library_finder.py:56:27: E741 ambiguous variable name 'l'
pythonforandroid/recipes/reportlab/__init__.py:25:21: E741 ambiguous variable name 'l'

The recent `flake8==3.8.0` update reported new errors:
```
pythonforandroid/graph.py:133:27: E741 ambiguous variable name 'l'
pythonforandroid/toolchain.py:219:25: E741 ambiguous variable name 'l'
pythonforandroid/toolchain.py:1146:19: F522 '...'.format(...) has unused named argument(s): Fore
pythonforandroid/pythonpackage.py:690:23: F523 '...'.format(...) has unused arguments at position(s): 0
pythonforandroid/logger.py:202:36: E741 ambiguous variable name 'l'
pythonforandroid/logger.py:204:36: E741 ambiguous variable name 'l'
pythonforandroid/archs.py:150:25: E741 ambiguous variable name 'l'
pythonforandroid/recipes/android/src/android/_ctypes_library_finder.py:56:27: E741 ambiguous variable name 'l'
pythonforandroid/recipes/reportlab/__init__.py:25:21: E741 ambiguous variable name 'l'
```
Copy link
Member

@opacam opacam left a comment

Choose a reason for hiding this comment

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

👍

Thanks!!

@AndreMiras AndreMiras merged commit c70432d into kivy:develop May 12, 2020
@AndreMiras AndreMiras deleted the feature/linter branch May 12, 2020 07:16
lerela added a commit to chronolife-rd/python-for-android that referenced this pull request Jun 23, 2020
🔖 v2020.06.02

* Adds missing requests sub dependencies (kivy#2221)
* Bumps to Gradle 6.4.1 (kivy#2222)
* Bumps to Cython==0.29.19 (kivy#2220)
* Updates install and troubleshooting docs (kivy#2219)
* Bumps to Ubuntu 20.04 (kivy#2218)
* Attempt to improve the issue template (kivy#2217)
* Add `opencv_extras` recipe (kivy#2209)
* Split logic for build modes & debug symbols (kivy#2213)
* Troubleshoot SSL error (kivy#2205)
* Remove superfluous recipes fixes (kivy#2202)
* Add tests for hostpython3 recipe (kivy#2196)
* Fix for 'cannot find setuptools module' (kivy#2195)
* Rename `Hostpython3Recipe` class to camel case (kivy#2194)
* Fix `test_should_build` (kivy#2193)
* Add initial tests for python3 recipe (kivy#2192)
* PythonActivityUtil helper for unpacking data (kivy#2189)
* Fixes flake8 errors post update (kivy#2191)
* Share PythonUtil.java between bootstraps (kivy#2188)
* Java code linting using PMD 6.23.0 (kivy#2187)
* Deletes deprecated renpy Python{Activity,Service}.java (kivy#2186)
* Removes java concurrency/ folder (kivy#2185)
* Reuse common AssetExtract.java (kivy#2182)
* Use common Hardware.java (kivy#2183)
* Moves kamranzafar/ java directory to common/ (kivy#2184)
* Updates release documentation (kivy#2177)
* Fixes service only unittest loading (kivy#2181)
* Narrows some context manager scopes (kivy#2179)
* Downgrades to SDL2 2.0.9 (kivy#2180)
* Bump to SDL2 2.0.10 & extract .java from SDL2 tarball (kivy#2113)
* Adds pygame recipe (kivy#2164)
* Adds macOS install instructions (2165)
* Removed python2 support mention from README (kivy#2162)
* Adding more assets (kivy#2132)
* Get --add-source working for dirs in Gradle builds (kivy#2156)
* Fixes python build with macOS venv (kivy#2159)
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.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.