Yuvipanda has submitted this change and it was merged.

Change subject: Support repositories field being just a dictionary
......................................................................


Support repositories field being just a dictionary

Way too many things have just a dictionary instead of a list
of dictionaries

Change-Id: Ic7b86b37c2057314be6d142c7c5913f94158c728
---
M generate.py
1 file changed, 12 insertions(+), 11 deletions(-)

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



diff --git a/generate.py b/generate.py
index 232df99..82d7abb 100644
--- a/generate.py
+++ b/generate.py
@@ -31,17 +31,18 @@
         'homepage': package.get('homepage', None),
         'keywords': package.get('keywords', []),
     }
-    if 'repositories' in package and len(package['repositories']):
-        for repo in package['repositories']:
-            print package['name'], repo
-            if 'github.com/' in repo['url']:
-                parts = repo['url'].split('/')
-                user_name, repo_name = parts[3], parts[4]
-                if repo_name.endswith('.git'):
-                    repo_name = repo_name[:-4]
-                print 'Fetching starcount for %s/%s' % (user_name, repo_name)
-                lib['stars'] = github_stars(user_name, repo_name)
-                break
+    if 'repositories' in package:
+        if isinstance(package['repositories'], dict):
+            url = package['repositories']['url']
+        elif isinstance(package['repositories'], list) and 
len(package['repositories']) > 0:
+            url = package['repositories'][0]['url']
+        if 'github.com/' in url:
+            parts = url.split('/')
+            user_name, repo_name = parts[3], parts[4]
+            if repo_name.endswith('.git'):
+                repo_name = repo_name[:-4]
+            print 'Fetching starcount for %s/%s' % (user_name, repo_name)
+            lib['stars'] = github_stars(user_name, repo_name)
 
     libraries.append(lib)
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic7b86b37c2057314be6d142c7c5913f94158c728
Gerrit-PatchSet: 3
Gerrit-Project: labs/tools/cdnjs-index
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>

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

Reply via email to