# HG changeset patch
# User Goffi <goffi@goffi.org>
# Date 1605189196 -3600
# Node ID f30b238d9c45fb3071e09c4a3a87ed5e121af366
# Parent  55f9a38864af752ee262aa11711e6c51f4b8bb13
jp: follow best practices and use a dash (`-`) instead of underscore (`_`) for long options.

diff -r 55f9a38864af -r f30b238d9c45 sat_frontends/jp/cmd_file.py
--- a/sat_frontends/jp/cmd_file.py	Thu Nov 12 14:53:16 2020 +0100
+++ b/sat_frontends/jp/cmd_file.py	Thu Nov 12 14:53:16 2020 +0100
@@ -456,7 +456,7 @@
 
     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(
diff -r 55f9a38864af -r f30b238d9c45 sat_frontends/jp/cmd_message.py
--- a/sat_frontends/jp/cmd_message.py	Thu Nov 12 14:53:16 2020 +0100
+++ b/sat_frontends/jp/cmd_message.py	Thu Nov 12 14:53:16 2020 +0100
@@ -60,7 +60,7 @@
             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",
diff -r 55f9a38864af -r f30b238d9c45 sat_frontends/jp/cmd_pubsub.py
--- a/sat_frontends/jp/cmd_pubsub.py	Thu Nov 12 14:53:16 2020 +0100
+++ b/sat_frontends/jp/cmd_pubsub.py	Thu Nov 12 14:53:16 2020 +0100
@@ -1530,7 +1530,7 @@
         )
         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"),
diff -r 55f9a38864af -r f30b238d9c45 sat_frontends/jp/cmd_roster.py
--- a/sat_frontends/jp/cmd_roster.py	Thu Nov 12 14:53:16 2020 +0100
+++ b/sat_frontends/jp/cmd_roster.py	Thu Nov 12 14:53:16 2020 +0100
@@ -200,10 +200,10 @@
 
     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):