Exclude kwarg when None

Author: andrewrabert

git/cmd.py

@@ -885,15 +885,15 @@ def transform_kwarg(self, name, value, split_single_char_options):
 iflen(name) ==1:
 ifvalueisTrue:
 return ["-%s"%name]
-eliftype(value) isnotbool:
+elifvaluenotin (False, None):
 ifsplit_single_char_options:
 return ["-%s"%name, "%s"%value]
 else:
 return ["-%s%s"% (name, value)]
 else:
 ifvalueisTrue:
 return ["--%s"%dashify(name)]
-eliftype(value) isnotbool:
+elifvaluenotin (False, None):
 return ["--%s=%s"% (dashify(name), value)]
 return []
 

git/test/test_git.py

@@ -82,13 +82,15 @@ def test_it_raises_errors(self):
 deftest_it_transforms_kwargs_into_git_command_arguments(self):
 assert_equal(["-s"], self.git.transform_kwargs(**{'s': True}))
 assert_equal(["-s", "5"], self.git.transform_kwargs(**{'s': 5}))
+assert_equal([], self.git.transform_kwargs(**{'s': None}))
 
 assert_equal(["--max-count"], self.git.transform_kwargs(**{'max_count': True}))
 assert_equal(["--max-count=5"], self.git.transform_kwargs(**{'max_count': 5}))
+assert_equal([], self.git.transform_kwargs(**{'max_count': None}))
 
 # Multiple args are supported by using lists/tuples
 assert_equal(["-L", "1-3", "-L", "12-18"], self.git.transform_kwargs(**{'L': ('1-3', '12-18')}))
-assert_equal(["-C", "-C"], self.git.transform_kwargs(**{'C': [True, True]}))
+assert_equal(["-C", "-C"], self.git.transform_kwargs(**{'C': [True, True, None, False]}))
 
 # order is undefined
 res=self.git.transform_kwargs(**{'s': True, 't': True})