Merge pull request #13396 from Carreau/cleanup-tools

Cleanup tools

tools/gh_api.py

@@ -5,14 +5,12 b' try:'
 except NameError:
     pass
 
-import os
 import re
 import sys
 
 import requests
 import getpass
 import json
-from pathlib import Path
 
 try:
     import requests_cache
@@ -82,28 +80,6 b' def get_auth_token():'
 def make_auth_header():
     return {'Authorization': 'token ' + get_auth_token()}
 
-def post_issue_comment(project, num, body):
-    url = 'https://api.github.com/repos/{project}/issues/{num}/comments'.format(project=project, num=num)
-    payload = json.dumps({'body': body})
-    requests.post(url, data=payload, headers=make_auth_header())
-
-def post_gist(content, description='', filename='file', auth=False):
-    """Post some text to a Gist, and return the URL."""
-    post_data = json.dumps({
-      "description": description,
-      "public": True,
-      "files": {
-        filename: {
-          "content": content
-        }
-      }
-    }).encode('utf-8')
-
-    headers = make_auth_header() if auth else {}
-    response = requests.post("https://api.github.com/gists", data=post_data, headers=headers)
-    response.raise_for_status()
-    response_data = json.loads(response.text)
-    return response_data['html_url']
 
 def get_pull_request(project, num, auth=False):
     """get pull request info  by number
@@ -118,15 +94,6 b' def get_pull_request(project, num, auth=False):'
     response.raise_for_status()
     return json.loads(response.text, object_hook=Obj)
 
-def get_pull_request_files(project, num, auth=False):
-    """get list of files in a pull request"""
-    url = "https://api.github.com/repos/{project}/pulls/{num}/files".format(project=project, num=num)
-    if auth:
-        header = make_auth_header()
-    else:
-        header = None
-    return get_paged_request(url, headers=header)
-
 element_pat = re.compile(r'<(.+?)>')
 rel_pat = re.compile(r'rel=[\'"](\w+)[\'"]')
 
@@ -149,17 +116,6 b' def get_paged_request(url, headers=None, **params):'
             break
     return results
 
-def get_pulls_list(project, auth=False, **params):
-    """get pull request list"""
-    params.setdefault("state", "closed")
-    url = "https://api.github.com/repos/{project}/pulls".format(project=project)
-    if auth:
-        headers = make_auth_header()
-    else:
-        headers = None
-    pages = get_paged_request(url, headers=headers, **params)
-    return pages
-
 def get_issues_list(project, auth=False, **params):
     """get issues list"""
     params.setdefault("state", "closed")
@@ -205,100 +161,3 b' def get_authors(pr):'
         authors.append("%s <%s>" % (author['name'], author['email']))
     return authors
 
-# encode_multipart_formdata is from urllib3.filepost
-# The only change is to iter_fields, to enforce S3's required key ordering
-
-def iter_fields(fields):
-    fields = fields.copy()
-    for key in ('key', 'acl', 'Filename', 'success_action_status', 'AWSAccessKeyId',
-        'Policy', 'Signature', 'Content-Type', 'file'):
-        yield (key, fields.pop(key))
-    for (k,v) in fields.items():
-        yield k,v
-
-def encode_multipart_formdata(fields, boundary=None):
-    """
-    Encode a dictionary of ``fields`` using the multipart/form-data mime format.
-
-    :param fields:
-        Dictionary of fields or list of (key, value) field tuples.  The key is
-        treated as the field name, and the value as the body of the form-data
-        bytes. If the value is a tuple of two elements, then the first element
-        is treated as the filename of the form-data section.
-
-        Field names and filenames must be unicode.
-
-    :param boundary:
-        If not specified, then a random boundary will be generated using
-        :func:`mimetools.choose_boundary`.
-    """
-    # copy requests imports in here:
-    from io import BytesIO
-    from requests.packages.urllib3.filepost import (
-        choose_boundary, six, writer, b, get_content_type
-    )
-    body = BytesIO()
-    if boundary is None:
-        boundary = choose_boundary()
-
-    for fieldname, value in iter_fields(fields):
-        body.write(b('--%s\r\n' % (boundary)))
-
-        if isinstance(value, tuple):
-            filename, data = value
-            writer(body).write('Content-Disposition: form-data; name="%s"; '
-                               'filename="%s"\r\n' % (fieldname, filename))
-            body.write(b('Content-Type: %s\r\n\r\n' %
-                       (get_content_type(filename))))
-        else:
-            data = value
-            writer(body).write('Content-Disposition: form-data; name="%s"\r\n'
-                               % (fieldname))
-            body.write(b'Content-Type: text/plain\r\n\r\n')
-
-        if isinstance(data, int):
-            data = str(data)  # Backwards compatibility
-        if isinstance(data, six.text_type):
-            writer(body).write(data)
-        else:
-            body.write(data)
-
-        body.write(b'\r\n')
-
-    body.write(b('--%s--\r\n' % (boundary)))
-
-    content_type = b('multipart/form-data; boundary=%s' % boundary)
-
-    return body.getvalue(), content_type
-
-
-def post_download(project, filename, name=None, description=""):
-    """Upload a file to the GitHub downloads area"""
-    if name is None:
-        name = Path(filename).name
-    with open(filename, 'rb') as f:
-        filedata = f.read()
-
-    url = "https://api.github.com/repos/{project}/downloads".format(project=project)
-
-    payload = json.dumps(dict(name=name, size=len(filedata),
-                    description=description))
-    response = requests.post(url, data=payload, headers=make_auth_header())
-    response.raise_for_status()
-    reply = json.loads(response.content)
-    s3_url = reply['s3_url']
-
-    fields = dict(
-        key=reply['path'],
-        acl=reply['acl'],
-        success_action_status=201,
-        Filename=reply['name'],
-        AWSAccessKeyId=reply['accesskeyid'],
-        Policy=reply['policy'],
-        Signature=reply['signature'],
-        file=(reply['name'], filedata),
-    )
-    fields['Content-Type'] = reply['mime_type']
-    data, content_type = encode_multipart_formdata(fields)
-    s3r = requests.post(s3_url, data=data, headers={'Content-Type': content_type})
-    return s3r

tools/github_stats.py

@@ -10,7 +10,6 b' To generate a report for IPython 2.0, run:'
 #-----------------------------------------------------------------------------
 
 
-import codecs
 import sys
 
 from argparse import ArgumentParser

tools/release

@@ -12,7 +12,6 b' import sys'
 
 from toollib import (get_ipdir, pjoin, cd, execfile, sh, archive,
          archive_user, archive_dir)
-from gh_api import post_download
 
 # Get main ipython dir, this will raise if it doesn't pass some checks
 ipdir = get_ipdir()
@@ -53,12 +52,6 b" if 'upload' in sys.argv:"
 
     # do not upload OS specific files like .DS_Store
     to_upload = glob('*.whl')+glob('*.tar.gz')
-    for fname in to_upload:
-        # TODO: update to GitHub releases API
-        continue
-        print('uploading %s to GitHub' % fname)
-        desc = "IPython %s source distribution" % version
-        post_download("ipython/ipython", fname, description=desc)
 
     # Make target dir if it doesn't exist
     print('1. Uploading IPython to archive.ipython.org')