From c3b20f127a3a38b47505f06b1d25822bdbeb19e3 2012-06-29 22:33:12
From: MinRK <benjaminrk@gmail.com>
Date: 2012-06-29 22:33:12
Subject: [PATCH] exclude non-merged pulls from gh stats

---

diff --git a/tools/github_stats.py b/tools/github_stats.py
index 7490539..aaf6022 100755
--- a/tools/github_stats.py
+++ b/tools/github_stats.py
@@ -89,6 +89,11 @@ latter case, it is used as a time before the present."""
     allclosed = get_paged_request(url)
     # allclosed = get_issues(project=project, state='closed', pulls=pulls, since=period)
     filtered = [i for i in allclosed if _parse_datetime(i['closed_at']) > period]
+    
+    # exclude rejected PRs
+    if pulls:
+        filtered = [ pr for pr in filtered if pr['merged_at'] ]
+    
     return filtered
 
 
@@ -103,7 +108,7 @@ def report(issues, show_urls=False):
     # titles may have unicode in them, so we must encode everything below
     if show_urls:
         for i in issues:
-            role = 'ghpull' if 'merged' in i else 'ghissue'
+            role = 'ghpull' if 'merged_at' in i else 'ghissue'
             print('* :%s:`%d`: %s' % (role, i['number'],
                                         i['title'].encode('utf-8')))
     else: