Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

lint: Clean up E713 PEP-8 rule.

This commit is contained in:
Tim Abbott 2017-01-23 21:08:52 -08:00
parent a088cdaa04
commit e385b93448
4 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/check-templates
View File

@ -76,7 +76,7 @@ def check_html_templates(templates, modified_only, all_dups):
'register',
]
bad_ids_dict = {ids: fns for ids, fns in template_id_dict.items()
if (not ids in IGNORE_IDS) and len(fns) > 1}
if (ids not in IGNORE_IDS) and len(fns) > 1}
if all_dups:
ignorable_ids_dict = {ids: fns for ids, fns in template_id_dict.items()

2
tools/html-grep
View File

@ -77,7 +77,7 @@ def check_our_files():
ftypes=['handlebars', 'html'],
)
fns = [fn for fn in files if (not 'casperjs' in fn) and (not 'puppet/zulip' in fn)]
fns = [fn for fn in files if ('casperjs' not in fn) and ('puppet/zulip' not in fn)]
grep(fns, set(keywords))
if __name__ == '__main__':

2
tools/lint-all
View File

@ -88,7 +88,7 @@ def check_pep8(files):
# actually be cleaned up.
#
'E123', 'E126', 'E226', 'E241', 'E261', 'E302',
'E401', 'E501', 'E702', 'E711', 'E712', 'E713',
'E401', 'E501', 'E702', 'E711', 'E712',
#
# Each of these rules are ignored for the explained reason.

2
tools/minify-js
View File

@ -112,7 +112,7 @@ for js_group, filespec in six.iteritems(JS_SPECS):
map_file = os.path.join(MAP_DIR, os.path.basename(filespec['output_filename']) +
'.map')
if (not 'force_minify' in filespec) and \
if ('force_minify' not in filespec) and \
(prev_deploy and len(set(in_files) & changed_files) == 0):
# Try to reuse the output file from previous deploy
try: