fix: 🐛 Replicate check for ValidatedSoftware used with SoftwareVersio…

…n delete() in Meraki integration.

changes/648.fixed

@@ -1 +1,2 @@
-Fixed Citrix ADM deleting SoftwareVersion in use with ValidatedSoftware.
+Fixed Citrix ADM deleting SoftwareVersion in use with ValidatedSoftware.
+Fixed Meraki deleting SoftwareVersion in use with ValidatedSoftware.

nautobot_ssot/integrations/meraki/diffsync/models/nautobot.py

@@ -114,10 +114,16 @@ def create(cls, adapter, ids, attrs):
         return super().create(adapter=adapter, ids=ids, attrs=attrs)
 
     def delete(self):
-        """Delete DeviceType in Nautobot from NautobotHardware object."""
-        super().delete()
+        """Delete SoftwareVersion in Nautobot from NautobotOSVersion object."""
         osversion = SoftwareVersion.objects.get(id=self.uuid)
-        osversion.delete()
+        if hasattr(osversion, "validatedsoftwarelcm_set"):
+            if osversion.validatedsoftwarelcm_set.count() != 0:
+                self.adapter.job.logger.warning(
+                    f"SoftwareVersion {osversion.version} for {osversion.platform.name} is used with a ValidatedSoftware so won't be deleted."
+                )
+        else:
+            super().delete()
+            osversion.delete()
         return self
 
 

tasks.py

@@ -768,7 +768,8 @@ def pylint(context):
     else:
         print("No migrations directory found, skipping migrations checks.")
 
-    raise Exit(code=exit_code)
+    if exit_code == 1:
+        raise Exit(code=exit_code)
 
 
 @task(aliases=("a",))
@@ -812,7 +813,8 @@ def ruff(context, action=None, target=None, fix=False, output_format="concise"):
         if not run_command(context, command, warn=True):
             exit_code = 1
 
-    raise Exit(code=exit_code)
+    if exit_code == 1:
+        raise Exit(code=exit_code)
 
 
 @task