target: Make all Target.__init__() parameters after working_directory…

… keyword-only

Prevent Target argument soup and allow future non-breaking evolutions of the Target
API by restricting what parameters can be positional. Only
"connection_settings", "platform" and "working_directory" can now be
passed positionally, all the other ones must be passed as keyword
arguments.

devlib/target.py

@@ -312,6 +312,7 @@ def __init__(self,
                  connection_settings=None,
                  platform=None,
                  working_directory=None,
+                 *,
                  executables_directory=None,
                  tmp_directory=None,
                  connect=True,
@@ -1791,6 +1792,7 @@ def __init__(self,
                  connection_settings=None,
                  platform=None,
                  working_directory=None,
+                 *,
                  executables_directory=None,
                  tmp_directory=None,
                  connect=True,
@@ -2001,6 +2003,7 @@ def __init__(self,
                  connection_settings=None,
                  platform=None,
                  working_directory=None,
+                 *,
                  executables_directory=None,
                  tmp_directory=None,
                  connect=True,
@@ -3089,6 +3092,7 @@ def __init__(self,
                  connection_settings=None,
                  platform=None,
                  working_directory=None,
+                 *,
                  executables_directory=None,
                  tmp_directory=None,
                  connect=True,
@@ -3175,6 +3179,7 @@ def __init__(self,
                  connection_settings=None,
                  platform=None,
                  working_directory=None,
+                 *,
                  executables_directory=None,
                  android_working_directory=None,
                  android_executables_directory=None,