Skip to content

Commit d5aa156

Browse files
committed
Fix flake8 linting
1 parent 6de9d18 commit d5aa156

File tree

7 files changed

+40
-21
lines changed

7 files changed

+40
-21
lines changed

app/__init__.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@
77

88
try:
99
assert version_info >= (3, 7, 0)
10-
except AssertionError as e:
11-
print('Warning Current python version not supported')
12-
print('Current python version: {version}'.format(version=version_info))
10+
except AssertionError:
11+
print('Warning Current Python version not supported')
12+
print('Current Python version: {version}'.format(version=version_info))
1313
exit(1)
1414

1515

app/api/__init__.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from app.api import routes
21
from flask import Blueprint
32

43
bp = Blueprint('api', __name__)

app/api/routes.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ def get_resources():
2525
print(e)
2626

2727
finally:
28-
return jsonify([single_resource.serialize for single_resource in resources])
28+
return jsonify([single_resource.serialize for
29+
single_resource in resources])
2930

3031

3132
def get_languages():

app/cli.py

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,19 +27,23 @@ def import_resources(db):
2727
category_dict = {c.key(): c for c in categories_list}
2828
except AttributeError as e:
2929
print('-------> EXCEPTION OCCURED DURING DB SETUP')
30-
print('-------> Most likely you need to set the "SQLALCHEMY_DATABASE_URI"')
30+
print('-------> Most likely you need to set the '
31+
'"SQLALCHEMY_DATABASE_URI"')
3132
print(f'-------> Exception message: {e}')
3233
return
3334

3435
# Step 4: Create/Update each resource in the db_session
3536
for resource in unique_resources:
36-
resource['category'] = get_category(resource, category_dict) # Note: modifies the category_dict in place (bad?)
37+
# Note: modifies the category_dict in place (bad?)
38+
resource['category'] = get_category(resource, category_dict)
39+
# Note: modifies the language_dict in place (bad?)
3740
resource['languages'] = get_languages(resource,
38-
language_dict) # Note: modifies the language_dict in place (bad?)
41+
language_dict)
3942
existing_resource = existing_resources.get(resource['url'])
4043

4144
if existing_resource:
42-
resource == existing_resource or update_resource(resource, existing_resource)
45+
resource == existing_resource or \
46+
update_resource(resource, existing_resource)
4347
else:
4448
create_resource(resource, db)
4549

@@ -48,12 +52,10 @@ def import_resources(db):
4852
except exc.SQLAlchemyError as e:
4953
db.session.rollback()
5054
print('Flask SQLAlchemy Exception:', e)
51-
template = "An SQLAlchemy exception of type {0} occurred. Arguments:\n{1!r}"
5255
print(resource)
5356
except Exception as e:
5457
db.session.rollback()
5558
print('exception', e)
56-
template = "An exception of type {0} occurred. Arguments:\n{1!r}"
5759
print(resource)
5860

5961

@@ -65,7 +67,8 @@ def remove_duplicates(data):
6567
resource_dict[resource['url']] = True
6668
unique_resources.append(resource)
6769
else:
68-
print(f"Encountered a duplicate resource in resources.yml: {resource['url']}")
70+
print(f"Encountered a duplicate resource "
71+
f"in resources.yml: {resource['url']}")
6972
return unique_resources
7073

7174

@@ -127,7 +130,6 @@ def db_migrate():
127130
""" migration commands"""
128131
pass
129132

130-
131133
@db_migrate.command()
132134
def init():
133135
print(db)

app/health_check.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@ def health_database_status():
55
try:
66
db.session.query("1").from_statement("SELECT 1").all()
77
return '<h1>It works.</h1>'
8-
except:
8+
except Exception:
99
return '<h1>Something is broken.</h1>'

app/models.py

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,24 +2,37 @@
22
from sqlalchemy_utils import URLType
33

44
'''
5-
event_id = db.Column('event_id', db.Integer, db.ForeignKey('event.id'), nullable=False)
5+
event_id = db.Column('event_id',
6+
db.Integer,
7+
db.ForeignKey('event.id'),
8+
nullable=False)
69
event = db.relationship('Event')
7-
partner_id = db.Column('partner_id', db.Integer, db.ForeignKey('partner.id'))
10+
partner_id = db.Column('partner_id',
11+
db.Integer,
12+
db.ForeignKey('partner.id'))
813
partner = db.relationship('Partner')
914
1015
'''
1116

1217
language_identifier = db.Table('language_identifier',
13-
db.Column('resource_id', db.Integer, db.ForeignKey('resource.id')),
14-
db.Column('language_id', db.Integer, db.ForeignKey('language.id'))
18+
db.Column(
19+
'resource_id',
20+
db.Integer,
21+
db.ForeignKey('resource.id')),
22+
db.Column(
23+
'language_id',
24+
db.Integer,
25+
db.ForeignKey('language.id'))
1526
)
1627

1728

1829
class Resource(db.Model):
1930
id = db.Column(db.Integer, primary_key=True)
2031
name = db.Column(db.String, nullable=False)
2132
url = db.Column(URLType, nullable=False, unique=True)
22-
category_id = db.Column(db.Integer, db.ForeignKey('category.id'), nullable=False)
33+
category_id = db.Column(db.Integer,
34+
db.ForeignKey('category.id'),
35+
nullable=False)
2336
category = db.relationship('Category')
2437
languages = db.relationship('Language', secondary=language_identifier)
2538
paid = db.Column(db.Boolean, default=False)
@@ -72,7 +85,11 @@ def __hash__(self):
7285
return hash(self.url)
7386

7487
def __repr__(self):
75-
return f"<Resource \n\tName: {self.name}\n\tLanguages: {self.languages}\n\tCategory: {self.category}\n\tURL: {self.url}\n>"
88+
return (f"<Resource \n"
89+
f"\tName: {self.name}\n"
90+
f"\tLanguages: {self.languages}\n"
91+
f"\tCategory: {self.category}\n"
92+
f"\tURL: {self.url}\n>")
7693

7794

7895
class Category(db.Model):

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,4 @@ SQLAlchemy==1.2.8
66
SQLAlchemy-Utils==0.33.3
77
PyYAML==3.13
88
psycopg2-binary==2.7.5
9-
pylint==2.1.1
9+
flake8==3.5.0

0 commit comments

Comments
 (0)