Remove -o option from diff and commit (fixes #54)

Author: techtonik

gitless/cli/helpers.py

@@ -151,8 +151,8 @@ def __call__(self, parser, namespace, revs, option_string=None):
 
 def oei_flags(subparsers, repo):
   subparsers.add_argument(
-      '-o', '--only', nargs='+',
-      help='use only files given (files must be tracked modified or untracked)',
+      'only', nargs='*',
+      help='use only files given (tracked modified or untracked)',
       action=PathProcessor, repo=repo, metavar='file')
   subparsers.add_argument(
       '-e', '--exclude', nargs='+',
@@ -204,7 +204,7 @@ def _oei_validate(only, exclude, include, curr_b):
   """
   if only and (exclude or include):
     pprint.err(
-        'You provided a list of filenames to be committed only (-o) but also '
+        'You provided a list of filenames to be committed but also '
         'provided a list of files to be excluded (-e) or included (-i)')
     return False
 

gitless/tests/test_e2e.py

@@ -353,7 +353,7 @@ def setUp(self):
     super(TestDiffFile, self).setUp()
     utils.write_file(self.TRACKED_FP)
     utils.write_file(self.DIR_TRACKED_FP)
-    gl.commit('-o', self.TRACKED_FP, self.DIR_TRACKED_FP, m='commit')
+    gl.commit(self.TRACKED_FP, self.DIR_TRACKED_FP, m='commit')
     utils.write_file(self.UNTRACKED_FP)
 
   def test_empty_diff(self):
@@ -472,7 +472,7 @@ def test_only_one(self):
         self.__build(self.OTHER, cids=[0]) + self.__build('master'))
 
   def test_only_some(self):
-    gl.fuse(self.OTHER, '-o', self.commits[self.OTHER][:2])
+    gl.fuse(self.OTHER, self.commits[self.OTHER][:2])
     self.__assert_history(
         self.__build(self.OTHER, [0, 1]) + self.__build('master'))