Skip to content

Commit 7bb7e3f

Browse files
committed
fix lint
1 parent 5db4d64 commit 7bb7e3f

File tree

6 files changed

+27
-38
lines changed

6 files changed

+27
-38
lines changed

.gitignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,4 +62,4 @@ ENV/
6262

6363
# Spyder project settings
6464
.spyderproject
65-
.idea/*
65+
.idea/*

inoreader/client.py

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424

2525

2626
class InoreaderClient(object):
27-
2827
# paths
2928
TOKEN_PATH = '/oauth2/token'
3029
USER_INFO_PATH = 'user-info'
@@ -190,13 +189,11 @@ def fetch_articles(self, folder=None, tags=None, unread=True, starred=False, lim
190189
fetched_count = 0
191190
response = self.parse_response(self.session.post(url, params=params, proxies=self.proxies))
192191
for data in response['items']:
193-
categories = set(
194-
[
195-
category.split('/')[-1]
196-
for category in data.get('categories', [])
197-
if category.find('label') > 0
198-
]
199-
)
192+
categories = {
193+
category.split('/')[-1]
194+
for category in data.get('categories', [])
195+
if category.find('label') > 0
196+
}
200197
if tags and not categories.issuperset(set(tags)):
201198
continue
202199

@@ -212,13 +209,11 @@ def fetch_articles(self, folder=None, tags=None, unread=True, starred=False, lim
212209
self.session.post(url, params=params, proxies=self.proxies)
213210
)
214211
for data in response['items']:
215-
categories = set(
216-
[
217-
category.split('/')[-1]
218-
for category in data.get('categories', [])
219-
if category.find('label') > 0
220-
]
221-
)
212+
categories = {
213+
category.split('/')[-1]
214+
for category in data.get('categories', [])
215+
if category.find('label') > 0
216+
}
222217
if tags and not categories.issuperset(set(tags)):
223218
continue
224219
yield Article.from_json(data)
@@ -305,6 +300,6 @@ def edit_subscription(self, stream_id, action, title=None, add_folder=None, remo
305300
response = self.parse_response(
306301
r,
307302
# self.session.post(url, params=params, proxies=self.proxies),
308-
json_data=False
303+
json_data=False,
309304
)
310305
return response

inoreader/main.py

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ def wrapper(*args, **kwargs):
9595
return wrapper
9696

9797

98-
@click.group(context_settings=dict(help_option_names=['-h', '--help']))
98+
@click.group(context_settings={'help_option_names': ['-h', '--help']})
9999
def main():
100100
pass
101101

@@ -362,7 +362,7 @@ def get_subscriptions(outfile, folder, out_format):
362362
client = get_client()
363363
results = []
364364
for sub in client.get_subscription_list():
365-
sub_categories = set([category['label'] for category in sub.categories])
365+
sub_categories = {category['label'] for category in sub.categories}
366366
if folder and folder not in sub_categories:
367367
continue
368368

@@ -452,9 +452,7 @@ def dedupe(folder, thresh):
452452
for docid, doc, _ in related:
453453
if docid == article.id:
454454
continue
455-
sims[doc] = sim_of(
456-
doc, article.title, method='cosine', term='char', ngram_range=(2, 3)
457-
)
455+
sims[doc] = sim_of(doc, article.title, method='cosine', term='char', ngram_range=(2, 3))
458456

459457
if sims and max(sims.values()) >= thresh:
460458
top_doc, top_score = sims.most_common()[0]
@@ -587,10 +585,13 @@ def fetch_starred(folder, tags, outfile, outdir, limit, save_image, out_format):
587585

588586

589587
@main.command("edit-subscription")
590-
@click.option("-a", "--action",
591-
required=True,
592-
type=click.Choice(['follow', 'unfollow', 'rename', 'add-folder', 'remove-folder']),
593-
help="")
588+
@click.option(
589+
"-a",
590+
"--action",
591+
required=True,
592+
type=click.Choice(['follow', 'unfollow', 'rename', 'add-folder', 'remove-folder']),
593+
help="",
594+
)
594595
@click.option("-i", "--stream-id", required=True, help='Stream ID which you want to fetch')
595596
@click.option("-n", "--name", help='The name of subscription, for action follow/rename(required)')
596597
@click.option("-f", "--folder", help='Folder which subscription belong to')
@@ -616,11 +617,7 @@ def edit_subscriptions(action, stream_id, name, folder):
616617
remove_folder = folder if action == 'remove-folder' else None
617618
try:
618619
response = client.edit_subscription(
619-
stream_id,
620-
edit_action,
621-
title=name,
622-
add_folder=add_folder,
623-
remove_folder=remove_folder
620+
stream_id, edit_action, title=name, add_folder=add_folder, remove_folder=remove_folder
624621
)
625622
click.secho(response, fg="green")
626623
except Exception as exception:

inoreader/sim.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ def cosine_sim(
9696
first_norm += freq**2
9797
inner_product += freq * second_term_freq[term]
9898

99-
for term, freq in second_term_freq.items():
99+
for _, freq in second_term_freq.items():
100100
second_norm += freq**2
101101

102102
if first_norm == 0 and second_norm == 0:

inoreader/subscription.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def from_json(cls, data):
1818
subscription_info = {
1919
'id': data['id'],
2020
'title': data['title'],
21-
'categories': [item for item in data['categories']],
21+
'categories': list(data['categories']),
2222
'sortid': data['sortid'],
2323
'firstitemmsec': data['firstitemmsec'],
2424
'url': data['url'],

setup.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
#!/usr/bin/env python
22
# coding: utf-8
33

4-
from setuptools import setup, find_packages
5-
4+
from setuptools import find_packages, setup
65

76
VERSION = '0.5.0'
87
REQS = [
@@ -25,7 +24,5 @@
2524
install_requires=REQS,
2625
include_package_data=True,
2726
zip_safe=False,
28-
entry_points={
29-
'console_scripts': ['inoreader=inoreader.main:main']
30-
},
27+
entry_points={'console_scripts': ['inoreader=inoreader.main:main']},
3128
)

0 commit comments

Comments
 (0)