LBRYCommunity/kivy-ios
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

🐛 fixes flake8 errors post update

Browse source 
The recent `flake8==3.8.1` update reported new errors:
```
kivy_ios/toolchain.py:292:31: E741 ambiguous variable name 'l'
...
```
Also note the entire `xcassets.py` file is being ignored as we can't
seem to use one ignore for the entire list anymore. I'm not sure if
this is a regression or a new feature from flake8.
This commit is contained in:
Andre Miras 2020-05-13 08:00:56 +02:00
parent 46647de39e
commit 4f1dffc754
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
kivy_ios/toolchain.py
View file

@ -289,7 +289,7 @@ class Graph:
graph = dict((k, set(v)) for k, v in self.graph.items()) graph = dict((k, set(v)) for k, v in self.graph.items())
while graph: while graph:
# Find all items without a parent # Find all items without a parent
leftmost = [l for l, s in graph.items() if not s] leftmost = [name for name, dep in graph.items() if not dep]
if not leftmost: if not leftmost:
raise ValueError('Dependency cycle detected! %s' % graph) raise ValueError('Dependency cycle detected! %s' % graph)
# If there is more than one, sort them for predictable order # If there is more than one, sort them for predictable order

5
kivy_ios/tools/external/xcassets.py vendored
View file

@ -4,6 +4,7 @@ Icon and LaunchImage generator for iOS
.. author:: Mathieu Virbel <mat@meltingrocks.com> .. author:: Mathieu Virbel <mat@meltingrocks.com>
""" """
# flake8: noqa (E121 mainly)
__all__ = ["launchimage"] __all__ = ["launchimage"]
@ -14,7 +15,7 @@ from os.path import join, exists
from os import makedirs from os import makedirs
appicon_json = { appicon_json = {
"images": [ # noqa: E121 "images": [
{ {
"idiom": "iphone", "idiom": "iphone",
"size": "20x20", "size": "20x20",
@ -338,7 +339,7 @@ appicon_json = {
launchimage_json = { launchimage_json = {
"images": [ # noqa: E121 "images": [
{ {
"extent": "full-screen", "extent": "full-screen",
"idiom": "iphone", "idiom": "iphone",