Milimetric has submitted this change and it was merged.

Change subject: Don't say generating graph twice when generating full graphs
......................................................................


Don't say generating graph twice when generating full graphs

Change-Id: I3a98208dfcf9c63f565462d5dd1d1d8f3c4133f0
---
M generate.py
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Milimetric: Verified; Looks good to me, approved



diff --git a/generate.py b/generate.py
index ad1641b..75f15f1 100644
--- a/generate.py
+++ b/generate.py
@@ -154,7 +154,7 @@
             due_at = last_run_time + increment
 
             if due_at < now or self.force:
-                print "Generating %s"%key
+                print('Generating {0}'.format(value['title']))
                 if "timeboxed" in value and "starts" in value:
                     from_date = value["starts"]
 
@@ -239,7 +239,6 @@
             print 'Bad SQL given'
 
     def generate_graph_full( self, key, value ):
-            print('Generating {0}'.format(value['title']))
             # Look for the sql first, then python
             db_name = value.get('db', self.config['defaults']['db'])
 

-- 
To view, visit https://gerrit.wikimedia.org/r/109787
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3a98208dfcf9c63f565462d5dd1d1d8f3c4133f0
Gerrit-PatchSet: 1
Gerrit-Project: analytics/limn-mobile-data
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: Milimetric <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to