Merge pull request #130 from pubs/feat/tag_completion_in_add

Adds tag list completion in `pubs add -t `

.travis.yml

@@ -1,27 +1,43 @@
-os:
-  - linux
-language: python
-python:
-  - "2.7"
-  - "3.3"
-  - "3.4"
-  - "3.5"
-  - "3.6"
+# list of environments to test
 matrix:
   include:
-    - language: generic
-      os: osx
-      python:
-        - "3"
+    - os: linux
+      language: python
+      python: 2.7
+    - os: linux
+      language: python
+      python: 3.3
+    - os: linux
+      language: python
+      python: 3.4
+    - os: linux
+      language: python
+      python: 3.5
+    - os: linux
+      language: python
+      python: 3.6
+    - os: osx
+      language: generic
+      python: 2.7
+      before_install:
+        - python2 --version
+        - pip2 install -U virtualenv
+        - virtualenv env -p python2
+        - source env/bin/activate
+    - os: osx
+      language: generic
+      python: ">=3.6"
       before_install:
         - brew update
         - brew outdated python3 || brew install python3 || brew upgrade python3
-        - pip install -U virtualenv
-        - virtualenv env -p python3
+        - python3 -m venv env
         - source env/bin/activate
+
 # command to install dependencies
 install:
-    - "pip install -r tests/requirements.txt"
-    - "python setup.py install"
+    - python --version
+    - pip install -r tests/requirements.txt
+    - python setup.py install
+
 # command to run tests
 script: python -m unittest discover

pubs/commands/add_cmd.py

@@ -6,9 +6,9 @@ from .. import repo
 from .. import paper
 from .. import templates
 from .. import apis
-from .. import color
 from .. import pretty
 from .. import utils
+from ..completion import CommaSeparatedTagsCompletion
 
 
 class ValidateDOI(argparse.Action):
@@ -26,7 +26,8 @@ def parser(subparsers, conf):
     parser.add_argument('-I', '--isbn', help='isbn number to retrieve the bibtex entry, if it is not provided', default=None)
     parser.add_argument('-d', '--docfile', help='pdf or ps file', default=None)
     parser.add_argument('-t', '--tags', help='tags associated to the paper, separated by commas',
-                        default=None)
+                        default=None
+                        ).completer = CommaSeparatedTagsCompletion(conf)
     parser.add_argument('-k', '--citekey', help='citekey associated with the paper;\nif not provided, one will be generated automatically.',
                         default=None)
     parser.add_argument('-L', '--link', action='store_false', dest='copy', default=True,

pubs/completion.py

@@ -59,6 +59,11 @@ class TagModifierCompletion(BaseCompleter):
         return [partial_expr + t for t in tags if t.startswith(t_prefix)]
 
 
+class CommaSeparatedTagsCompletion(TagModifierCompletion):
+
+    regxp = r"[^,]*$"
+
+
 class CommaSeparatedListCompletion(BaseCompleter):
 
     values = []

tests/requirements.txt

@@ -1,4 +1,4 @@
 # those are the additional packages required to run the tests
 six
-pyfakefs
+pyfakefs==3.3
 ddt