First pip installable release

Added setup.py MANIFEST.in
Added more things to do in TODO.md
Refactored gifc to handle more exceptions
In update api, change description and file arguements can be used independently
Removed the need for config.yml file. Now we extract data from environment variables
Create now uses whatever editor you mention amongst [nano, vi, gedit]
remove files recognizes when you pass files that are not present

Author: armsp

MANIFEST.in

@@ -0,0 +1 @@
+include README.md

TODO.md

@@ -136,7 +136,26 @@ else:
     print('Creating gist failed')
 ```
 9. - [ ] Get method doesn't work for private gists. May have to use [OAuth](https://developer.github.com/apps/building-oauth-apps/authorizing-oauth-apps/) for that.
+11. - [ ] Requests raises exceptions inherited from RequestException that you are not catching. So commands like `r = request.post(....)` can fail on bad internet connections or because other random issues. Catch them like -  
+        ```python
+        except requests.exceptions.RequestException as e:  # This is the correct syntax
+            print e
+        ```
+        OR
+        ```python
+        except requests.exceptions.Timeout:
+            # Maybe set up for a retry, or continue in a retry loop
+        except requests.exceptions.TooManyRedirects:
+            # Tell the user their URL was bad and try a different one
+        except requests.exceptions.RequestException as e:
+            # catastrophic error. bail.
+            print e
+        ```
+12. Handle FileNotFoundError in creating of gists via file parameter.
 10. Instead of using `__file__` as in `foo_config = open(os.path.join(os.path.dirname(__file__),'foo.conf').read()` or ```cwd = os.path.dirname(os.path.abspath(__file__))```, use `pkg_resources` instead. [Help1](http://peak.telecommunity.com/DevCenter/PythonEggs#accessing-package-resources), [Help2](https://setuptools.readthedocs.io/en/latest/pkg_resources.html#resourcemanager-api)
+13. Displaying gists in cli
+14. Update is still hardcoded with `nano`. Can we open it in system default editor?
+15. If no file arguement and description is given in update then open the first file in that gist in interactive mode for editing ?
 ## It is NOT meant to replace the GUI neither does it attempt to.
 
 ## Features we will never have

gifc

@@ -1,12 +1,12 @@
 #!/usr/bin/env python3
-# -*- coding: utf-8 -*-
-import argparse
-import requests
-import yaml
+
+import os
 import pprint
-import subprocess
+import argparse
 import tempfile
-import os
+import subprocess
+
+import requests
 
 parser = argparse.ArgumentParser(description='Github gists from command line')
 sp = parser.add_subparsers(dest='main')
@@ -27,9 +27,9 @@ p2_mutex.add_argument('-i','--interactive', nargs='?', action='store', choices=[
 #Update gists
 p3 = sp.add_parser('update')
 p3.add_argument('gist_id', help='Gist id(sha) of the gist you want to update')
-p3.add_argument('-f','--file_name', required=True, help='File name of the file inside the gist you are updating')
+p3.add_argument('-f','--file_name', nargs='?', action='store', help='File name of the file inside the gist you are updating')
 #parser.add_argument('-nc','--new_content', help='New content string')
-p3.add_argument('-cd','--change_description', help='Change the gist description')
+p3.add_argument('-cd','--change_description', nargs='?', action='store', help='Change the gist description')
 #Delete gists
 p4 = sp.add_parser('delete')
 p4.add_argument('gist_id', help='Gist id(sha) of the gist you want to delete')
@@ -41,28 +41,17 @@ p5.add_argument('-r', '--remove', nargs='+', required=True, help='File(s) you wa
 args = parser.parse_args()
 
 def get_config_details(key):
-    try:
-        with open('gist_config.yml', 'r') as f:
-            config = yaml.safe_load(f)
-    except:
-        print('Configuration file not found')
-        exit()
+    if key in os.environ:
+        return os.environ[key]
     else:
-        if config.get(key):
-            return config[key]
-        else:
-            if key == 'TOKEN':
-                print('User token not found. Please add your token to gist_config.yaml')
-                exit()
-            elif key == 'USER_ID':
-                print('User ID not found. Please add your github user_id to gist_config.yml')
-                exit()
+        print('Environment Variable not set. Please set and try again')
+        exit()
 
 class Color:
-    red = '\x1b[1;31;40m'
+    red = '\x1b[1;31;47m'
     green = '\x1b[1;32;40m'
     yellow = '\x1b[1;33;40m'
-    white = '\x1b[1;37;40m'
+    white = '\x1b[1;30;50m'
     bold = '\033[1m'
     end = '\x1b[0m'
 
@@ -73,12 +62,12 @@ def pretty_print(gist_list):
         print('Gist ID    : ', Color.white+gist['Gist ID']+Color.end)
         print('><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><')
 
-
+# Get particular gist   https://api.github.com/gists/aa5a315d61ae9438b18d
 #Get gists
 if args.main == 'get':
     number_of_gists = args.number
     print('Getting gists...')
-    user_id = get_config_details('USER_ID')
+    user_id = get_config_details('GIT_USER_ID')
     r = requests.get(f'https://api.github.com/users/{user_id}/gists')
     gist_list = []
     for gist in r.json()[:number_of_gists]:
@@ -93,7 +82,7 @@ if args.main == 'get':
 if args.main == 'create':
     #print(args['create'])
     gist_file = args.gist_item
-    header = {"Authorization": f"Bearer {get_config_details('TOKEN')}"}
+    header = {"Authorization": f"Bearer {get_config_details('GIT_TOKEN')}"}
 
     if args.message:
         message = args.message
@@ -119,15 +108,25 @@ if args.main == 'create':
         else:
             print('Error creating gist. Please use GUI')
     else:
+        try:
+            check_editor = subprocess.Popen([args.interactive])
+            check_editor.terminate()
+        except:
+            print(Color.red+'Could not open that editor. Please use another.'+Color.end)
+            exit()
         f = tempfile.NamedTemporaryFile(mode='w+t', delete=False)
         f_name = f.name
         f.close()
-        subprocess.run(['nano', f_name])
+        subprocess.run([args.interactive, f_name])
         with open(f_name) as f:
             contents = f.read()
 
         payload = {'description': args.describe, 'public': args.public, 'files':{args.gist_item : {'content': contents}}}
-        r = requests.post(f"https://api.github.com/gists", headers=header, json=payload)
+        try:
+            r = requests.post(f"https://api.github.com/gists", headers=header, json=payload)
+        except requests.exceptions.RequestException as e:
+            print(e)
+            exit()
         if r.status_code == 201:
             print(r.status_code)
             print(r.json()['id'])
@@ -139,7 +138,7 @@ if args.main == 'create':
 #Update gists
 if args.main == 'update':
     gid = args.gist_id
-    header = {"Authorization": f"Bearer {get_config_details('TOKEN')}"}
+    header = {"Authorization": f"Bearer {get_config_details('GIT_TOKEN')}"}
     payload = dict()
     #file content changes
     if args.file_name:
@@ -173,7 +172,7 @@ if args.main == 'update':
 if args.main == 'delete':
     gid = args.gist_id
     print(f'Deleting {gid}')
-    token = get_config_details('TOKEN')
+    token = get_config_details('GIT_TOKEN')
     header = {'Authorization': f'Bearer {token}'}
 
     r = requests.delete(f'https://api.github.com/gists/{gid}', headers=header)
@@ -187,15 +186,18 @@ if args.main == 'remove':
     gid = args.gist_id
     files = args.remove
     pprint.pprint(f'Deleting file(s) {files}')
-    token = get_config_details('TOKEN')
+    token = get_config_details('GIT_TOKEN')
     header = {'Authorization': f'Bearer {token}'}
     #print(files, len(files))
     file_del_payload = dict(zip(files,[None]*len(files)))
     payload = {"files": file_del_payload}
-    print(payload)
+    #print(payload)
     r = requests.patch(f'https://api.github.com/gists/{gid}', headers=header, json=payload)
     #data vs json when you have to pass "null" as json.
-    if r.status_code != 200:
+    if r.status_code == 422:
+        print('You are trying to delete a file that does not exist. Please try again with correct file.')
+        exit()
+    elif r.status_code != 200:
         print('Request not completed. Please try the GUI.')
         exit()
     print('File(s) deleted.')

setup.py

@@ -0,0 +1,42 @@
+from setuptools import setup, find_packages
+import sys
+
+if sys.version_info[0] != 3:
+    raise RuntimeError('Unsupported python version "{0}"'.format(sys.version_info[0]))
+
+try:
+    with open('README.md') as f:
+        long_description = f.read()
+except:
+    long_description = 'This is a nifty little tool written in Python to work with GitHub Gists from command line.'
+
+setup(name='gifc',
+      version='0.1',
+      description='Github Gists from CLI',
+      long_description=long_description,
+      long_description_content_type='text/markdown',
+      keywords=['gist', 'github', 'cli'],
+      author='Shantam Raj',
+      author_email='[email protected]',
+      url='https://github.com/armsp/gifc',
+      download_url='',
+      include_package_data=True,
+      packages=find_packages('gifc'),
+      scripts=['gifc'],
+      license='GNU GPL v3',
+      install_requires=[
+          'requests',
+          ],
+      python_requires=">=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*",
+      classifiers=[
+            'Development Status :: 3 - Alpha',
+            'Intended Audience :: Developers',
+            'License :: OSI Approved :: GNU General Public License v3 or later (GPLv3+)',
+            'Natural Language :: English',
+            'Programming Language :: Python :: 3.4',
+            'Programming Language :: Python :: 3.5',
+            'Programming Language :: Python :: 3.6',
+            'Programming Language :: Python :: 3.7',
+            ],
+      zip_safe=False
+      )