jp: follow best practices and use a dash (-) instead of underscore (_) for long options.

sat_frontends/jp/cmd_file.py

@@ -456,7 +456,7 @@ class Get(base.CommandBase):
 
     def add_parser_options(self):
         self.parser.add_argument(
-            '-o', '--dest_file', type=str, default='',
+            '-o', '--dest-file', type=str, default='',
             help=_("destination file (DEFAULT: filename from URL)")
             )
         self.parser.add_argument(

sat_frontends/jp/cmd_message.py

@@ -60,7 +60,7 @@ class Send(base.CommandBase):
             help=_("subject of the message"),
         )
         self.parser.add_argument(
-            "-L", "--subject_lang", type=str, default="", help=_("language of subject")
+            "-L", "--subject-lang", type=str, default="", help=_("language of subject")
         )
         self.parser.add_argument(
             "-t",

sat_frontends/jp/cmd_pubsub.py

@@ -1530,7 +1530,7 @@ class Transform(base.CommandBase):
         )
         self.parser.add_argument(
             "-I",
-            "--ignore_errors",
+            "--ignore-errors",
             action="store_true",
             help=_(
                 "if command return a non zero exit code, ignore the item and continue"),

sat_frontends/jp/cmd_roster.py

@@ -200,10 +200,10 @@ class Purge(base.CommandBase):
 
     def add_parser_options(self):
         self.parser.add_argument(
-            "--no_from", action="store_true",
+            "--no-from", action="store_true",
             help=_("also purge contacts with no 'from' subscription"))
         self.parser.add_argument(
-            "--no_to", action="store_true",
+            "--no-to", action="store_true",
             help=_("also purge contacts with no 'to' subscription"))
 
     async def start(self):