Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: sitescripts/management/bin/generateNotifications.py

Issue 29756624: Noissue - Adapt best practices for trailing commas (sitescripts) (Closed)
Patch Set: Re-run script on Python 2, added flake8-commas extension Created April 19, 2018, 2:35 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 # This file is part of the Adblock Plus web scripts, 1 # This file is part of the Adblock Plus web scripts,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
(...skipping 10 matching lines...) Expand all
21 from sitescripts.utils import get_config, setupStderr 21 from sitescripts.utils import get_config, setupStderr
22 22
23 23
24 def generate_notifications(path): 24 def generate_notifications(path):
25 notifications = load_notifications() 25 notifications = load_notifications()
26 # Ignoring notifications with variants here - we can only process those in a 26 # Ignoring notifications with variants here - we can only process those in a
27 # URL handler. 27 # URL handler.
28 notifications = [x for x in notifications if 'variants' in x] 28 notifications = [x for x in notifications if 'variants' in x]
29 output = { 29 output = {
30 'notifications': notifications, 30 'notifications': notifications,
31 'version': time.strftime('%Y%m%d%H%M', time.gmtime()) 31 'version': time.strftime('%Y%m%d%H%M', time.gmtime()),
32 } 32 }
33 with codecs.open(path, 'wb', encoding='utf-8') as file: 33 with codecs.open(path, 'wb', encoding='utf-8') as file:
34 json.dump(output, file, ensure_ascii=False, indent=2, 34 json.dump(output, file, ensure_ascii=False, indent=2,
35 separators=(',', ': '), sort_keys=True) 35 separators=(',', ': '), sort_keys=True)
36 36
37 37
38 if __name__ == '__main__': 38 if __name__ == '__main__':
39 setupStderr() 39 setupStderr()
40 output = get_config().get('notifications', 'output') 40 output = get_config().get('notifications', 'output')
41 generate_notifications(output) 41 generate_notifications(output)
OLDNEW
« no previous file with comments | « sitescripts/management/bin/generateHgAuth.py ('k') | sitescripts/management/bin/installChanges.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld