refactor(Commit): Added deprecation on git signoff mechanic

Signoff dedicated code is no longer necessary & removed #TODO as tests show identical args and extra-args are not an issue.

Author: SCrocky

commitizen/cli.py

@@ -398,7 +398,6 @@ def main():
             raise InvalidCommandArgumentError(
                 f"Invalid commitizen arguments were found before -- separator: `{' '.join(unknown_args[:pos])}`. "
             )
-        # TODO: treat case when extra-args and commitizen args are identical
         extra_args = " ".join(unknown_args[1:])
         arguments["extra_cli_args"] = extra_args
 

commitizen/commands/commit.py

@@ -82,6 +82,7 @@ def __call__(self):
         signoff: bool = self.arguments.get("signoff")
 
         if signoff:
+            out.warn("signoff mechanic is deprecated, please use `cz commit -- -s` instead.")
             c = git.commit(m, "-s")
 
         if self.arguments.get("extra_cli_args"):