Merge pull request #8 from jatindhankhar/update-base-image

Bump docker base image to fedora 26 and fix url split issue.
jatindhankhar · Jul 26, 2017 · 4f7e816 · 4f7e816
2 parents 02a7541 + 0801228
commit 4f7e816
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/activity-build-docker/Dockerfile b/activity-build-docker/Dockerfile
@@ -1,4 +1,4 @@
-FROM fedora:22
+FROM fedora:26
 
 RUN dnf install -y git gettext sugar-toolkit-gtk3 sugar-toolkit
 

diff --git a/aslo/api/gh.py b/aslo/api/gh.py
@@ -1,7 +1,6 @@
 import hmac
 import hashlib
 from flask import current_app as app
-from urllib.parse import urlparse
 from github import Github
 
 
@@ -16,11 +15,8 @@ def auth():
 
 
 def get_developers(repo_url):
-    o = urlparse(repo_url)
-    repo = o.path[1:].strip('.git')
-
     g = auth()
-    repository = g.get_repo(repo)
+    repository = g.get_repo(repo_url)
     contributors = repository.get_contributors()
     developers = []
     for c in contributors:

diff --git a/aslo/api/release.py b/aslo/api/release.py
@@ -316,7 +316,7 @@ def handle_release(gh_json):
         metadata['i18n_summary'] = {'en': metadata.get('summary', '')}
 
     metadata['repository'] = repo_url
-    metadata['developers'] = gh.get_developers(metadata['repository'])
+    metadata['developers'] = gh.get_developers(gh_json['repository']['full_name'])
     metadata['icon_bin'] = img.get_icon(repo_path, metadata['icon'])
 
     try: