Merged in hotfix/publish-plugin-improvements (pull request #377)

Publishing plugin improvements

Approved-by: Milan Kolar <milan@orbi.tools>

pype/plugins/global/publish/cleanup.py

@@ -40,6 +40,15 @@ class CleanUp(pyblish.api.InstancePlugin):
     active = True
 
     def process(self, instance):
+        # Get the errored instances
+        failed = []
+        for result in instance.context.data["results"]:
+            if (result["error"] is not None and result["instance"] is not None
+               and result["instance"] not in failed):
+                failed.append(result["instance"])
+        assert instance not in failed, ("Result of '{}' instance "
+            "were not success".format(instance.data["name"]))
+
         if [ef for ef in self.exclude_families
                 if instance.data["family"] in ef]:
             return

pype/plugins/global/publish/integrate_new.py

@@ -62,7 +62,6 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin):
                 "render",
                 "imagesequence",
                 "review",
-                "render",
                 "rendersetup",
                 "rig",
                 "plate",

pype/plugins/nuke/publish/increment_script_version.py

@@ -14,7 +14,7 @@ class IncrementScriptVersion(pyblish.api.ContextPlugin):
     def process(self, context):
 
         assert all(result["success"] for result in context.data["results"]), (
-            "Atomicity not held, aborting.")
+            "Publishing not succesfull so version is not increased.")
 
         instances = context[:]