fix(pre-commit): It should not "fix" .svg files

The update_vehicle_templates.py script should not
produce files that conflict with the pre-comit rules

Author: amilcarlucas

.pre-commit-config.yaml

@@ -7,8 +7,8 @@ repos:
     rev: v5.0.0
     hooks:
       - id: end-of-file-fixer
-      - id: trailing-whitespace
         exclude: \.svg$
+      - id: trailing-whitespace
       - id: check-added-large-files
       - id: check-case-conflict
       - id: check-executables-have-shebangs

ardupilot_methodic_configurator/backend_filesystem_json_with_schema.py

@@ -9,7 +9,7 @@
 """
 
 from json import JSONDecodeError
-from json import dump as json_dump
+from json import dumps as json_dumps
 from json import load as json_load
 
 # from logging import warning as logging_warning
@@ -125,8 +125,10 @@ def save_json_data(self, data: dict, data_dir: str) -> tuple[bool, str]:  # noqa
         filepath = os_path.join(data_dir, self.json_filename)
         try:
             with open(filepath, "w", encoding="utf-8", newline="\n") as file:
-                json_dump(data, file, indent=4)
-                file.write("\n")  # pre-commit recommends adding an extra newline at the end of the file
+                json_str = json_dumps(data, indent=4)
+                # Strip the last newline to avoid double newlines
+                # This is to ensure compatibility with pre-commit's end-of-file-fixer
+                file.write(json_str.rstrip("\n") + "\n")
         except FileNotFoundError:
             msg = _("Directory '{}' not found").format(data_dir)
             logging_error(msg)