From e7aa413038f186b4f523318762d438f33c2004a8 Mon Sep 17 00:00:00 2001
From: Petr Kalis <petr.kalis@gmail.com>
Date: Thu, 4 May 2023 12:16:58 +0200
Subject: [PATCH 01/15] AfterEffects: add review flag to each instance (#4884)

* OP-5657 - add artist control for review in AfterEffects

Artist can disable review to be created for particular publish.

* OP-5657 - add artist control for review in AfterEffects

Removed configuration for Deadline, should be controlled by what is on instance.

* OP-5657 - handle legacy instances

Legacy instances wont't have mark_for_review in creator_attributes. Set to true as by default we always want review.

* OP-5657 - remove explicit review for all AE

Now handled directly on instance

* OP-5657 - fix - cannot remove now

Without this 'review' wont be added to tags on representation. Eventually this should be refactored.
Control on whole instance, eg. disabling review, should be enough.

* OP-5657 - fix - correct host name used

* OP-5657 - fix - correct handling of review

On local renders review should be added only from families, not from older approach through Settings.

Farm instance cannot have review in families or extract_review would get triggered even locally.

* OP-5657 - refactor - changed label

* OP-5657 - Hound

* OP-5657 - added explicitly skipping review

Instance might have set 'review' to False, which should explicitly skip review (might come from Publisher where artist can disable/enable review on an instance).

* OP-5657 - updated setting of review variable

instance.data.review == False >> explicitly set to do not create review. Keep None to let logic decide.

* OP-5657 - fix adding review flag

* OP-5657 - updated test

Removed review for second instance.

* OP-5657 - refactor to context plugin

* OP-5657 - tie thumbnail to review for local render

Produce thumbnail only when review should be created to synchronize state with farm rendering.
Move creation of thumnbail out of this plugin to general plugin to limit duplication of logic.
---
 .../plugins/create/create_render.py           | 45 +++++++++++++------
 .../plugins/publish/collect_render.py         | 18 +++-----
 .../plugins/publish/collect_review.py         | 25 +++++++++++
 .../plugins/publish/extract_local_render.py   | 28 +-----------
 .../plugins/publish/submit_publish_job.py     | 23 +++++++---
 .../publish/abstract_collect_render.py        |  2 +-
 .../project_settings/aftereffects.json        |  6 ++-
 .../schema_project_aftereffects.json          | 23 +++++++++-
 ...ublish_in_aftereffects_multicomposition.py |  9 ++--
 9 files changed, 116 insertions(+), 63 deletions(-)
 create mode 100644 openpype/hosts/aftereffects/plugins/publish/collect_review.py

diff --git a/openpype/hosts/aftereffects/plugins/create/create_render.py b/openpype/hosts/aftereffects/plugins/create/create_render.py
index c20b0ec51b..171d7053ce 100644
--- a/openpype/hosts/aftereffects/plugins/create/create_render.py
+++ b/openpype/hosts/aftereffects/plugins/create/create_render.py
@@ -26,12 +26,9 @@ class RenderCreator(Creator):
 
     create_allow_context_change = True
 
-    def __init__(self, project_settings, *args, **kwargs):
-        super(RenderCreator, self).__init__(project_settings, *args, **kwargs)
-        self._default_variants = (project_settings["aftereffects"]
-                                                  ["create"]
-                                                  ["RenderCreator"]
-                                                  ["defaults"])
+    # Settings
+    default_variants = []
+    mark_for_review = True
 
     def create(self, subset_name_from_ui, data, pre_create_data):
         stub = api.get_stub()  # only after After Effects is up
@@ -82,28 +79,40 @@ class RenderCreator(Creator):
                 use_farm = pre_create_data["farm"]
                 new_instance.creator_attributes["farm"] = use_farm
 
+            review = pre_create_data["mark_for_review"]
+            new_instance.creator_attributes["mark_for_review"] = review
+
             api.get_stub().imprint(new_instance.id,
                                    new_instance.data_to_store())
             self._add_instance_to_context(new_instance)
 
             stub.rename_item(comp.id, subset_name)
 
-    def get_default_variants(self):
-        return self._default_variants
-
-    def get_instance_attr_defs(self):
-        return [BoolDef("farm", label="Render on farm")]
-
     def get_pre_create_attr_defs(self):
         output = [
             BoolDef("use_selection", default=True, label="Use selection"),
             BoolDef("use_composition_name",
                     label="Use composition name in subset"),
             UISeparatorDef(),
-            BoolDef("farm", label="Render on farm")
+            BoolDef("farm", label="Render on farm"),
+            BoolDef(
+                "mark_for_review",
+                label="Review",
+                default=self.mark_for_review
+            )
         ]
         return output
 
+    def get_instance_attr_defs(self):
+        return [
+            BoolDef("farm", label="Render on farm"),
+            BoolDef(
+                "mark_for_review",
+                label="Review",
+                default=False
+            )
+        ]
+
     def get_icon(self):
         return resources.get_openpype_splash_filepath()
 
@@ -143,6 +152,13 @@ class RenderCreator(Creator):
                 api.get_stub().rename_item(comp_id,
                                            new_comp_name)
 
+    def apply_settings(self, project_settings, system_settings):
+        plugin_settings = (
+            project_settings["aftereffects"]["create"]["RenderCreator"]
+        )
+
+        self.mark_for_review = plugin_settings["mark_for_review"]
+
     def get_detail_description(self):
         return """Creator for Render instances
 
@@ -201,4 +217,7 @@ class RenderCreator(Creator):
             instance_data["creator_attributes"] = {"farm": is_old_farm}
             instance_data["family"] = self.family
 
+        if instance_data["creator_attributes"].get("mark_for_review") is None:
+            instance_data["creator_attributes"]["mark_for_review"] = True
+
         return instance_data
diff --git a/openpype/hosts/aftereffects/plugins/publish/collect_render.py b/openpype/hosts/aftereffects/plugins/publish/collect_render.py
index 6153a426cf..b01b707246 100644
--- a/openpype/hosts/aftereffects/plugins/publish/collect_render.py
+++ b/openpype/hosts/aftereffects/plugins/publish/collect_render.py
@@ -88,10 +88,11 @@ class CollectAERender(publish.AbstractCollectRender):
                 raise ValueError("No file extension set in Render Queue")
             render_item = render_q[0]
 
+            instance_families = inst.data.get("families", [])
             subset_name = inst.data["subset"]
             instance = AERenderInstance(
                 family="render",
-                families=inst.data.get("families", []),
+                families=instance_families,
                 version=version,
                 time="",
                 source=current_file,
@@ -109,6 +110,7 @@ class CollectAERender(publish.AbstractCollectRender):
                 tileRendering=False,
                 tilesX=0,
                 tilesY=0,
+                review="review" in instance_families,
                 frameStart=frame_start,
                 frameEnd=frame_end,
                 frameStep=1,
@@ -139,6 +141,9 @@ class CollectAERender(publish.AbstractCollectRender):
                 instance.toBeRenderedOn = "deadline"
                 instance.renderer = "aerender"
                 instance.farm = True  # to skip integrate
+                if "review" in instance.families:
+                    # to skip ExtractReview locally
+                    instance.families.remove("review")
 
             instances.append(instance)
             instances_to_remove.append(inst)
@@ -218,15 +223,4 @@ class CollectAERender(publish.AbstractCollectRender):
         if fam not in instance.families:
             instance.families.append(fam)
 
-        settings = get_project_settings(os.getenv("AVALON_PROJECT"))
-        reviewable_subset_filter = (settings["deadline"]
-                                    ["publish"]
-                                    ["ProcessSubmittedJobOnFarm"]
-                                    ["aov_filter"].get(self.hosts[0]))
-        for aov_pattern in reviewable_subset_filter:
-            if re.match(aov_pattern, instance.subset):
-                instance.families.append("review")
-                instance.review = True
-                break
-
         return instance
diff --git a/openpype/hosts/aftereffects/plugins/publish/collect_review.py b/openpype/hosts/aftereffects/plugins/publish/collect_review.py
new file mode 100644
index 0000000000..a933b9fed2
--- /dev/null
+++ b/openpype/hosts/aftereffects/plugins/publish/collect_review.py
@@ -0,0 +1,25 @@
+"""
+Requires:
+    None
+
+Provides:
+    instance     -> family ("review")
+"""
+import pyblish.api
+
+
+class CollectReview(pyblish.api.ContextPlugin):
+    """Add review to families if instance created with 'mark_for_review' flag
+    """
+    label = "Collect Review"
+    hosts = ["aftereffects"]
+    order = pyblish.api.CollectorOrder + 0.1
+
+    def process(self, context):
+        for instance in context:
+            creator_attributes = instance.data.get("creator_attributes") or {}
+            if (
+                creator_attributes.get("mark_for_review")
+                and "review" not in instance.data["families"]
+            ):
+                instance.data["families"].append("review")
diff --git a/openpype/hosts/aftereffects/plugins/publish/extract_local_render.py b/openpype/hosts/aftereffects/plugins/publish/extract_local_render.py
index d535329eb4..c70aa41dbe 100644
--- a/openpype/hosts/aftereffects/plugins/publish/extract_local_render.py
+++ b/openpype/hosts/aftereffects/plugins/publish/extract_local_render.py
@@ -66,33 +66,9 @@ class ExtractLocalRender(publish.Extractor):
             first_repre = not representations
             if instance.data["review"] and first_repre:
                 repre_data["tags"] = ["review"]
+                thumbnail_path = os.path.join(staging_dir, files[0])
+                instance.data["thumbnailSource"] = thumbnail_path
 
             representations.append(repre_data)
 
         instance.data["representations"] = representations
-
-        ffmpeg_path = get_ffmpeg_tool_path("ffmpeg")
-        # Generate thumbnail.
-        thumbnail_path = os.path.join(staging_dir, "thumbnail.jpg")
-
-        args = [
-            ffmpeg_path, "-y",
-            "-i", first_file_path,
-            "-vf", "scale=300:-1",
-            "-vframes", "1",
-            thumbnail_path
-        ]
-        self.log.debug("Thumbnail args:: {}".format(args))
-        try:
-            output = run_subprocess(args)
-        except TypeError:
-            self.log.warning("Error in creating thumbnail")
-            six.reraise(*sys.exc_info())
-
-        instance.data["representations"].append({
-            "name": "thumbnail",
-            "ext": "jpg",
-            "files": os.path.basename(thumbnail_path),
-            "stagingDir": staging_dir,
-            "tags": ["thumbnail"]
-        })
diff --git a/openpype/modules/deadline/plugins/publish/submit_publish_job.py b/openpype/modules/deadline/plugins/publish/submit_publish_job.py
index f80bd40133..eeb813cb62 100644
--- a/openpype/modules/deadline/plugins/publish/submit_publish_job.py
+++ b/openpype/modules/deadline/plugins/publish/submit_publish_job.py
@@ -438,7 +438,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             "Finished copying %i files" % len(resource_files))
 
     def _create_instances_for_aov(
-        self, instance_data, exp_files, additional_data
+        self, instance_data, exp_files, additional_data, do_not_add_review
     ):
         """Create instance for each AOV found.
 
@@ -449,6 +449,8 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             instance_data (pyblish.plugin.Instance): skeleton data for instance
                 (those needed) later by collector
             exp_files (list): list of expected files divided by aovs
+            additional_data (dict):
+            do_not_add_review (bool): explicitly skip review
 
         Returns:
             list of instances
@@ -514,8 +516,6 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
 
             app = os.environ.get("AVALON_APP", "")
 
-            preview = False
-
             if isinstance(col, list):
                 render_file_name = os.path.basename(col[0])
             else:
@@ -532,6 +532,8 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             new_instance = deepcopy(instance_data)
             new_instance["subset"] = subset_name
             new_instance["subsetGroup"] = group_name
+
+            preview = preview and not do_not_add_review
             if preview:
                 new_instance["review"] = True
 
@@ -591,7 +593,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             self.log.debug("instances:{}".format(instances))
         return instances
 
-    def _get_representations(self, instance, exp_files):
+    def _get_representations(self, instance, exp_files, do_not_add_review):
         """Create representations for file sequences.
 
         This will return representations of expected files if they are not
@@ -602,6 +604,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             instance (dict): instance data for which we are
                              setting representations
             exp_files (list): list of expected files
+            do_not_add_review (bool): explicitly skip review
 
         Returns:
             list of representations
@@ -651,6 +654,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             if instance.get("slate"):
                 frame_start -= 1
 
+            preview = preview and not do_not_add_review
             rep = {
                 "name": ext,
                 "ext": ext,
@@ -705,6 +709,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             preview = match_aov_pattern(
                 host_name, self.aov_filter, remainder
             )
+            preview = preview and not do_not_add_review
             if preview:
                 rep.update({
                     "fps": instance.get("fps"),
@@ -820,8 +825,12 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
         families = [family]
 
         # pass review to families if marked as review
+        do_not_add_review = False
         if data.get("review"):
             families.append("review")
+        elif data.get("review") == False:
+            self.log.debug("Instance has review explicitly disabled.")
+            do_not_add_review = True
 
         instance_skeleton_data = {
             "family": family,
@@ -977,7 +986,8 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             instances = self._create_instances_for_aov(
                 instance_skeleton_data,
                 data.get("expectedFiles"),
-                additional_data
+                additional_data,
+                do_not_add_review
             )
             self.log.info("got {} instance{}".format(
                 len(instances),
@@ -986,7 +996,8 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
         else:
             representations = self._get_representations(
                 instance_skeleton_data,
-                data.get("expectedFiles")
+                data.get("expectedFiles"),
+                do_not_add_review
             )
 
             if "representations" not in instance_skeleton_data.keys():
diff --git a/openpype/pipeline/publish/abstract_collect_render.py b/openpype/pipeline/publish/abstract_collect_render.py
index ccb2415346..fd35ddb719 100644
--- a/openpype/pipeline/publish/abstract_collect_render.py
+++ b/openpype/pipeline/publish/abstract_collect_render.py
@@ -58,7 +58,7 @@ class RenderInstance(object):
     # With default values
     # metadata
     renderer = attr.ib(default="")  # renderer - can be used in Deadline
-    review = attr.ib(default=False)  # generate review from instance (bool)
+    review = attr.ib(default=None)  # False - explicitly skip review
     priority = attr.ib(default=50)  # job priority on farm
 
     family = attr.ib(default="renderlayer")
diff --git a/openpype/settings/defaults/project_settings/aftereffects.json b/openpype/settings/defaults/project_settings/aftereffects.json
index 669e1db0b8..6128534344 100644
--- a/openpype/settings/defaults/project_settings/aftereffects.json
+++ b/openpype/settings/defaults/project_settings/aftereffects.json
@@ -13,10 +13,14 @@
         "RenderCreator": {
             "defaults": [
                 "Main"
-            ]
+            ],
+            "mark_for_review": true
         }
     },
     "publish": {
+        "CollectReview": {
+            "enabled": true
+        },
         "ValidateSceneSettings": {
             "enabled": true,
             "optional": true,
diff --git a/openpype/settings/entities/schemas/projects_schema/schema_project_aftereffects.json b/openpype/settings/entities/schemas/projects_schema/schema_project_aftereffects.json
index 8dc83f5506..313e0ce8ea 100644
--- a/openpype/settings/entities/schemas/projects_schema/schema_project_aftereffects.json
+++ b/openpype/settings/entities/schemas/projects_schema/schema_project_aftereffects.json
@@ -40,7 +40,13 @@
                             "label": "Default Variants",
                             "object_type": "text",
                             "docstring": "Fill default variant(s) (like 'Main' or 'Default') used in subset name creation."
-                        }
+                        },
+                        {
+                            "type": "boolean",
+                            "key": "mark_for_review",
+                            "label": "Review",
+                            "default": true
+                         }
                     ]
                 }
             ]
@@ -51,6 +57,21 @@
             "key": "publish",
             "label": "Publish plugins",
             "children": [
+                {
+                    "type": "dict",
+                    "collapsible": true,
+                    "key": "CollectReview",
+                    "label": "Collect Review",
+                    "checkbox_key": "enabled",
+                    "children": [
+                        {
+                            "type": "boolean",
+                            "key": "enabled",
+                            "label": "Enabled",
+                            "default": true
+                        }
+                    ]
+                },
                 {
                     "type": "dict",
                     "collapsible": true,
diff --git a/tests/integration/hosts/aftereffects/test_deadline_publish_in_aftereffects_multicomposition.py b/tests/integration/hosts/aftereffects/test_deadline_publish_in_aftereffects_multicomposition.py
index d372efcb9a..0e9cd3b00d 100644
--- a/tests/integration/hosts/aftereffects/test_deadline_publish_in_aftereffects_multicomposition.py
+++ b/tests/integration/hosts/aftereffects/test_deadline_publish_in_aftereffects_multicomposition.py
@@ -9,6 +9,9 @@ log = logging.getLogger("test_publish_in_aftereffects")
 class TestDeadlinePublishInAfterEffectsMultiComposition(AEDeadlinePublishTestClass):  # noqa
     """est case for DL publishing in AfterEffects with multiple compositions.
 
+        Workfile contains 2 prepared `render` instances. First has review set,
+        second doesn't.
+
         Uses generic TestCase to prepare fixtures for test data, testing DBs,
         env vars.
 
@@ -68,7 +71,7 @@ class TestDeadlinePublishInAfterEffectsMultiComposition(AEDeadlinePublishTestCla
                                     name="renderTest_taskMain2"))
 
         failures.append(
-            DBAssert.count_of_types(dbcon, "representation", 7))
+            DBAssert.count_of_types(dbcon, "representation", 5))
 
         additional_args = {"context.subset": "workfileTest_task",
                            "context.ext": "aep"}
@@ -105,13 +108,13 @@ class TestDeadlinePublishInAfterEffectsMultiComposition(AEDeadlinePublishTestCla
         additional_args = {"context.subset": "renderTest_taskMain2",
                            "name": "thumbnail"}
         failures.append(
-            DBAssert.count_of_types(dbcon, "representation", 1,
+            DBAssert.count_of_types(dbcon, "representation", 0,
                                     additional_args=additional_args))
 
         additional_args = {"context.subset": "renderTest_taskMain2",
                            "name": "png_exr"}
         failures.append(
-            DBAssert.count_of_types(dbcon, "representation", 1,
+            DBAssert.count_of_types(dbcon, "representation", 0,
                                     additional_args=additional_args))
 
         assert not any(failures)

From cb24f2a9ba8c08b27980fa06e2cb83a2599173a9 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 14:07:19 +0200
Subject: [PATCH 02/15] :art: configure linter

---
 .github/workflows/pr_linting.yml | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 .github/workflows/pr_linting.yml

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
new file mode 100644
index 0000000000..1deeef4726
--- /dev/null
+++ b/.github/workflows/pr_linting.yml
@@ -0,0 +1,23 @@
+name: 📇 Code Linting
+
+on:
+  push:
+    branches: [ develop ]
+  pull_request:
+    branches: [ develop ]
+
+permissions:
+  contents: read
+  pull-requests: write
+
+jobs:
+  linting:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - name: Code Check
+        uses: wemake-services/wemake-python-styleguide@master
+        with:
+          reporter: 'github-pr-review'
+        env:
+          GITHUB_TOKEN: "${{ secrets.YNPUT_BOT_TOKEN }}"

From e608e7c808c22a09c118b4acad14c66543342b7e Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 14:57:28 +0200
Subject: [PATCH 03/15] :art: limit to changed files path

---
 .github/workflows/pr_linting.yml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 1deeef4726..1f6be64d31 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -15,9 +15,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
+      - run: |
+          echo "_CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }} -- '*.py' | tr -s '\n' ' ' )" >> ${GITHUB_ENV}
       - name: Code Check
         uses: wemake-services/wemake-python-styleguide@master
         with:
           reporter: 'github-pr-review'
+          path: "${{ env._CHANGED_FILES }}"
         env:
           GITHUB_TOKEN: "${{ secrets.YNPUT_BOT_TOKEN }}"

From 5829f647e548ab46be77a4caf7d704297419bbf8 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 14:59:41 +0200
Subject: [PATCH 04/15] :art: limit

---
 .github/workflows/pr_linting.yml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 1f6be64d31..59ac48aeb8 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -6,6 +6,8 @@ on:
   pull_request:
     branches: [ develop ]
 
+  workflow_dispatch:
+
 permissions:
   contents: read
   pull-requests: write
@@ -19,6 +21,7 @@ jobs:
           echo "_CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }} -- '*.py' | tr -s '\n' ' ' )" >> ${GITHUB_ENV}
       - name: Code Check
         uses: wemake-services/wemake-python-styleguide@master
+        if: ${{ env._CHANGED_FILES }}
         with:
           reporter: 'github-pr-review'
           path: "${{ env._CHANGED_FILES }}"

From 6fb6762f34c0b972e5061b5bfcae1755a6bba895 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:14:17 +0200
Subject: [PATCH 05/15] :recycle: limit using diff_context

---
 .github/workflows/pr_linting.yml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 59ac48aeb8..64b6bc0da5 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -17,13 +17,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
-      - run: |
-          echo "_CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }} -- '*.py' | tr -s '\n' ' ' )" >> ${GITHUB_ENV}
       - name: Code Check
         uses: wemake-services/wemake-python-styleguide@master
-        if: ${{ env._CHANGED_FILES }}
         with:
           reporter: 'github-pr-review'
-          path: "${{ env._CHANGED_FILES }}"
+          filter: 'diff_context'
         env:
           GITHUB_TOKEN: "${{ secrets.YNPUT_BOT_TOKEN }}"

From da12ac16c56f4510309275ec764b708e1a62bdb0 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:32:50 +0200
Subject: [PATCH 06/15] :art: solving changed list

---
 .github/workflows/pr_linting.yml | 38 +++++++++++++++++++++++++++++++-
 1 file changed, 37 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 64b6bc0da5..975c83e677 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -8,19 +8,55 @@ on:
 
   workflow_dispatch:
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number}}
+  cancel-in-progress: true
+
 permissions:
   contents: read
   pull-requests: write
 
 jobs:
+  files_changed:
+    runs-on: ubuntu-latest
+    outputs:
+        changed_python: ${{ steps.changes.outputs.python }}
+  steps:
+    - uses: actions/checkout@master
+      if: github.event_name == 'push'
+    - uses: dorny/paths-filter@master
+      id: changes
+      with:
+        filters: |
+          cpp:
+            - ["lte/gateway/c/**", "orc8r/gateway/c/**", "lte/gateway/python/**"]
+          javascript:
+            - ["nms/**", "**/*.js"]
+          python:
+            - ["**/*.py"]
+          terraform:
+            - ["**/*.tf"]
+
   linting:
+    needs: files_changed
+    if: ${{ needs.files_changed.outputs.changed_python == 'true' }}
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
+        with:
+          ref: ${{ github.event.pull_request.head.ref }}
+          repository: ${{ github.event.pull_request.head.repo.full_name }}
+          fetch-depth: 0
+      - name: Get changed Python files
+        id: py-changes
+        run: |
+          echo "py_files_list=$(git diff --name-only --diff-filter=ACMRT \
+            ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }} \
+            | grep .py$ | xargs)" >> $GITHUB_OUTPUT
       - name: Code Check
         uses: wemake-services/wemake-python-styleguide@master
         with:
           reporter: 'github-pr-review'
-          filter: 'diff_context'
+          path: ${{ steps.py-changes.outputs.py_files_list }}
         env:
           GITHUB_TOKEN: "${{ secrets.YNPUT_BOT_TOKEN }}"

From e1a2792a7a8d774abf1777fa079938d71559e57e Mon Sep 17 00:00:00 2001
From: Ynbot <bot@ynput.io>
Date: Thu, 4 May 2023 13:39:59 +0000
Subject: [PATCH 07/15] [Automated] Release

---
 CHANGELOG.md        | 248 ++++++++++++++++++++++++++++++++++++++++++++
 openpype/version.py |   2 +-
 pyproject.toml      |   2 +-
 3 files changed, 250 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 16deaaa4fd..07c1e7d5fd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,6 +1,254 @@
 # Changelog
 
 
+## [3.15.6](https://github.com/ynput/OpenPype/tree/3.15.6)
+
+
+[Full Changelog](https://github.com/ynput/OpenPype/compare/3.15.5...3.15.6)
+
+### **🆕 New features**
+
+
+<details>
+<summary>Substance Painter Integration <a href="https://github.com/ynput/OpenPype/pull/4283">#4283</a></summary>
+
+<strong>This implements a part of #4205 by implementing a Substance Painter integration
+
+</strong>Status:
+- [x] Implement Host
+- [x] start substance with last workfile using `AddLastWorkfileToLaunchArgs` prelaunch hook
+- [x] Implement Qt tools
+- [x] Implement loaders
+- [x] Implemented a Set project mesh loader (this is relatively special case because a Project will always have exactly one mesh - a Substance Painter project cannot exist without a mesh).
+- [x] Implement project open callback
+- [x] On project open it notifies the user if the loaded model is outdated
+- [x] Implement publishing logic
+- [x] Workfile publishing
+- [x] Export Texture Sets
+- [x] Support OCIO using #4195 (draft brach is set up - see comment)
+- [ ] Likely needs more testing on the OCIO front
+- [x] Validate all outputs of the Export template are exported/generated
+- [x] Allow validation to be optional **(issue: there's no API method to detect what maps will be exported without doing an actual export to disk)**
+- [x] Support extracting/integration if not all outputs are generated
+- [x] Support multiple materials/texture sets per instance
+- [ ] Add validator that can enforce only a single texture set output if studio prefers that.
+- [ ] Implement Export File Format (extensions) override in Creator
+- [ ] Add settings so Admin can choose which extensions are available.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Data Exchange: Geometry in 3dsMax <a href="https://github.com/ynput/OpenPype/pull/4555">#4555</a></summary>
+
+<strong>Introduces and updates a creator, extractors and loaders for model family
+
+</strong>Introduces new creator, extractors and loaders for model family while adding model families into the existing max scene loader and extractor
+- [x] creators
+- [x]  adding model family into max scene loader and extractor
+- [x]  fbx loader
+- [x]  fbx extractor
+- [x]  usd loader
+- [x]  usd extractor
+- [x] validator for model family
+- [x]  obj loader(update function)
+- [x]  fix the update function of the loader as #4675
+- [x]  Add documentation
+
+
+___
+
+</details>
+
+
+<details>
+<summary>AfterEffects: add review flag to each instance <a href="https://github.com/ynput/OpenPype/pull/4884">#4884</a></summary>
+
+Adds `mark_for_review` flag to the Creator to allow artists to disable review if necessary.Exposed this flag in Settings, by default set to True (eg. same behavior as previously).
+
+
+___
+
+</details>
+
+### **🚀 Enhancements**
+
+
+<details>
+<summary>Houdini: Fix Validate Output Node (VDB) <a href="https://github.com/ynput/OpenPype/pull/4819">#4819</a></summary>
+
+- Removes plug-in that was a duplicate of this plug-in.
+- Optimize logging of many prims slightly
+- Fix error reporting like https://github.com/ynput/OpenPype/pull/4818 did
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Houdini: Add null node as output indicator when using TAB search <a href="https://github.com/ynput/OpenPype/pull/4834">#4834</a></summary>
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Houdini: Don't error in collect review if camera is not set correctly <a href="https://github.com/ynput/OpenPype/pull/4874">#4874</a></summary>
+
+Do not raise an error in collector when invalid path is set as camera path. Allow camera path to not be set correctly in review instance until validation so it's nicely shown in a validation report.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Project packager: Backup and restore can store only database <a href="https://github.com/ynput/OpenPype/pull/4879">#4879</a></summary>
+
+Pack project functionality have option to zip only project database without project files. Unpack project can skip project copy if the folder is not found.Added helper functions to `openpype.client.mongo` that can be also used for tests as replacement of mongo dump.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Houdini: ExtractOpenGL for Review instance not optional <a href="https://github.com/ynput/OpenPype/pull/4881">#4881</a></summary>
+
+Don't make ExtractOpenGL optional for review instance optional.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Publisher: Small style changes <a href="https://github.com/ynput/OpenPype/pull/4894">#4894</a></summary>
+
+Small changes in styles and form of publisher UI.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Houdini: Workfile icon in new publisher <a href="https://github.com/ynput/OpenPype/pull/4898">#4898</a></summary>
+
+Fix icon for the workfile instance in new publisher
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Fusion: Simplify creator icons code <a href="https://github.com/ynput/OpenPype/pull/4899">#4899</a></summary>
+
+Simplify code for setting the icons for the Fusion creators
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Enhancement: Fix PySide 6.5 support for loader <a href="https://github.com/ynput/OpenPype/pull/4900">#4900</a></summary>
+
+Fixes PySide 6.5 support in Loader.
+
+
+___
+
+</details>
+
+### **🐛 Bug fixes**
+
+
+<details>
+<summary>Maya: Validate Attributes <a href="https://github.com/ynput/OpenPype/pull/4917">#4917</a></summary>
+
+This plugin was broken due to bad fetching of data and wrong repair action.
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Fix: Locally copied version of last published workfile is not incremented <a href="https://github.com/ynput/OpenPype/pull/4722">#4722</a></summary>
+
+### Fix 1
+When copied, the local workfile version keeps the published version number, when it must be +1 to follow OP's naming convention.
+
+### Fix 2
+Local workfile version's name is built from anatomy. This avoids to get workfiles with their publish template naming.
+
+### Fix 3
+In the case a subset has at least two tasks with published workfiles, for example `Modeling` and `Rigging`, launching `Rigging` was getting the first one with the `next` and trying to find representations, therefore `workfileModeling` and trying to match the current `task_name` (`Rigging`) with the `representation["context"]["task"]["name"]` of a Modeling representation, which was ending up to a `workfile_representation` to `None`, and exiting the process.
+
+Trying to find the `task_name` in the `subset['name']` fixes it.
+
+### Fix 4
+Fetch input dependencies of workfile.
+
+Replacing https://github.com/ynput/OpenPype/pull/4102 for changes to bring this home.
+___
+
+</details>
+
+
+<details>
+<summary>Maya: soft-fail when pan/zoom locked on camera when playblasting <a href="https://github.com/ynput/OpenPype/pull/4929">#4929</a></summary>
+
+When pan/zoom enabled attribute on camera is locked, playblasting with pan/zoom fails because it is trying to restore it. This is fixing it by skipping over with warning.
+
+
+___
+
+</details>
+
+### **Merged pull requests**
+
+
+<details>
+<summary>Maya Load References - Add Display Handle Setting <a href="https://github.com/ynput/OpenPype/pull/4904">#4904</a></summary>
+
+When we load a reference in Maya using OpenPype loader, display handle is checked by default and prevent us to select easily the object in the viewport. I understand that some productions like to keep this option, so I propose to add display handle to the reference loader settings. 
+
+
+___
+
+</details>
+
+
+<details>
+<summary>Photoshop: add autocreators for review and flat image <a href="https://github.com/ynput/OpenPype/pull/4871">#4871</a></summary>
+
+Review and flatten image (produced when no instance of `image` family was created) were created somehow magically. This PRintroduces two new auto creators which allow artists to disable review or flatten image.For all `image` instances `Review` flag was added to provide functionality to create separate review per `image` instance. Previously was possible only to have separate instance of `review` family.Review is not enabled on `image` family by default. (Eg. follows original behavior)Review auto creator is enabled by default as it was before.Flatten image creator must be set in Settings in `project_settings/photoshop/create/AutoImageCreator`.
+
+
+___
+
+</details>
+
+
+
+
 ## [3.15.5](https://github.com/ynput/OpenPype/tree/3.15.5)
 
 
diff --git a/openpype/version.py b/openpype/version.py
index 9832ff4747..dc0a3a8c9f 100644
--- a/openpype/version.py
+++ b/openpype/version.py
@@ -1,3 +1,3 @@
 # -*- coding: utf-8 -*-
 """Package declaring Pype version."""
-__version__ = "3.15.6-nightly.3"
+__version__ = "3.15.6"
diff --git a/pyproject.toml b/pyproject.toml
index 2f40d58f56..003f6cf2d3 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "OpenPype"
-version = "3.15.5" # OpenPype
+version = "3.15.6" # OpenPype
 description = "Open VFX and Animation pipeline with support."
 authors = ["OpenPype Team <info@openpype.io>"]
 license = "MIT License"

From 11343088b55eb9962a91919295ecbe865b03c11c Mon Sep 17 00:00:00 2001
From: "github-actions[bot]" <github-actions[bot]@users.noreply.github.com>
Date: Thu, 4 May 2023 13:41:12 +0000
Subject: [PATCH 08/15] chore(): update bug report / version

---
 .github/ISSUE_TEMPLATE/bug_report.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml
index 8328a35cad..5050d37c7a 100644
--- a/.github/ISSUE_TEMPLATE/bug_report.yml
+++ b/.github/ISSUE_TEMPLATE/bug_report.yml
@@ -35,6 +35,7 @@ body:
       label: Version
       description: What version are you running? Look to OpenPype Tray
       options:
+        - 3.15.6
         - 3.15.6-nightly.3
         - 3.15.6-nightly.2
         - 3.15.6-nightly.1
@@ -134,7 +135,6 @@ body:
         - 3.14.1-nightly.1
         - 3.14.0
         - 3.14.0-nightly.1
-        - 3.13.1-nightly.3
     validations:
       required: true
   - type: dropdown

From a3a996d5a56787fa88cf0d92a2c39a6bf0f31fbd Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:42:04 +0200
Subject: [PATCH 09/15] :bug: fix checkout action

---
 .github/workflows/pr_linting.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 975c83e677..4f4f7c2585 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -22,7 +22,7 @@ jobs:
     outputs:
         changed_python: ${{ steps.changes.outputs.python }}
   steps:
-    - uses: actions/checkout@master
+    - uses: actions/checkout@v3
       if: github.event_name == 'push'
     - uses: dorny/paths-filter@master
       id: changes

From 65110ded4609f2b6d7a5ccf97bff097f91d370de Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:43:32 +0200
Subject: [PATCH 10/15] :bug: fix checkout action

---
 .github/workflows/pr_linting.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 975c83e677..4f4f7c2585 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -22,7 +22,7 @@ jobs:
     outputs:
         changed_python: ${{ steps.changes.outputs.python }}
   steps:
-    - uses: actions/checkout@master
+    - uses: actions/checkout@v3
       if: github.event_name == 'push'
     - uses: dorny/paths-filter@master
       id: changes

From 5ef4f4fb131f569da9aa0b5db50c66c0a99f86e8 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:47:52 +0200
Subject: [PATCH 11/15] :bug: fix syntax

---
 .github/workflows/pr_linting.yml | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 4f4f7c2585..58f52bb313 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -21,21 +21,15 @@ jobs:
     runs-on: ubuntu-latest
     outputs:
         changed_python: ${{ steps.changes.outputs.python }}
-  steps:
-    - uses: actions/checkout@v3
-      if: github.event_name == 'push'
-    - uses: dorny/paths-filter@master
-      id: changes
-      with:
-        filters: |
-          cpp:
-            - ["lte/gateway/c/**", "orc8r/gateway/c/**", "lte/gateway/python/**"]
-          javascript:
-            - ["nms/**", "**/*.js"]
-          python:
-            - ["**/*.py"]
-          terraform:
-            - ["**/*.tf"]
+    steps:
+      - uses: actions/checkout@v3
+        if: github.event_name == 'push'
+      - uses: dorny/paths-filter@master
+        id: changes
+        with:
+          filters: |
+            python:
+              - ["**/*.py"]
 
   linting:
     needs: files_changed

From 3497b03b7fc8cffadf597908e7f4bdde60f1ee0e Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 15:50:49 +0200
Subject: [PATCH 12/15] :bug: fix syntax

---
 .github/workflows/pr_linting.yml | 24 +++++++++---------------
 1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/pr_linting.yml b/.github/workflows/pr_linting.yml
index 975c83e677..58f52bb313 100644
--- a/.github/workflows/pr_linting.yml
+++ b/.github/workflows/pr_linting.yml
@@ -21,21 +21,15 @@ jobs:
     runs-on: ubuntu-latest
     outputs:
         changed_python: ${{ steps.changes.outputs.python }}
-  steps:
-    - uses: actions/checkout@master
-      if: github.event_name == 'push'
-    - uses: dorny/paths-filter@master
-      id: changes
-      with:
-        filters: |
-          cpp:
-            - ["lte/gateway/c/**", "orc8r/gateway/c/**", "lte/gateway/python/**"]
-          javascript:
-            - ["nms/**", "**/*.js"]
-          python:
-            - ["**/*.py"]
-          terraform:
-            - ["**/*.tf"]
+    steps:
+      - uses: actions/checkout@v3
+        if: github.event_name == 'push'
+      - uses: dorny/paths-filter@master
+        id: changes
+        with:
+          filters: |
+            python:
+              - ["**/*.py"]
 
   linting:
     needs: files_changed

From 227075f1a918d3ddc7aea60a57f955558820171b Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 16:07:32 +0200
Subject: [PATCH 13/15] :art: add WPS to pre-commit

---
 .pre-commit-config.yaml |  9 +++++++++
 poetry.lock             | 24 +++++++++++++++++++++---
 pyproject.toml          |  1 +
 3 files changed, 31 insertions(+), 3 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index eec388924e..fe4c7e3da3 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -10,3 +10,12 @@ repos:
     -   id: check-added-large-files
     -   id: no-commit-to-branch
         args: [ '--pattern', '^(?!((release|enhancement|feature|bugfix|documentation|tests|local|chore)\/[a-zA-Z0-9\-_]+)$).*' ]
+- repo: local
+  hooks:
+      - id: flake8
+        name: flake8
+        description: WPS enforced flake8
+        entry: flake8
+        args: ["--config=setup.cfg"]
+        language: python
+        types: [python]
diff --git a/poetry.lock b/poetry.lock
index f71611cb6f..cbc4ac7b94 100644
--- a/poetry.lock
+++ b/poetry.lock
@@ -1,4 +1,4 @@
-# This file is automatically @generated by Poetry and should not be changed by hand.
+# This file is automatically @generated by Poetry 1.4.0 and should not be changed by hand.
 
 [[package]]
 name = "acre"
@@ -1456,11 +1456,13 @@ python-versions = ">=3.6"
 files = [
     {file = "lief-0.12.3-cp310-cp310-macosx_10_14_arm64.whl", hash = "sha256:66724f337e6a36cea1a9380f13b59923f276c49ca837becae2e7be93a2e245d9"},
     {file = "lief-0.12.3-cp310-cp310-macosx_10_14_x86_64.whl", hash = "sha256:6d18aafa2028587c98f6d4387bec94346e92f2b5a8a5002f70b1cf35b1c045cc"},
+    {file = "lief-0.12.3-cp310-cp310-macosx_11_0_arm64.whl", hash = "sha256:d4f69d125caaa8d5ddb574f29cc83101e165ebea1a9f18ad042eb3544081a797"},
     {file = "lief-0.12.3-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl", hash = "sha256:c078d6230279ffd3bca717c79664fb8368666f610b577deb24b374607936e9c1"},
     {file = "lief-0.12.3-cp310-cp310-win32.whl", hash = "sha256:e3a6af926532d0aac9e7501946134513d63217bacba666e6f7f5a0b7e15ba236"},
     {file = "lief-0.12.3-cp310-cp310-win_amd64.whl", hash = "sha256:0750b72e3aa161e1fb0e2e7f571121ae05d2428aafd742ff05a7656ad2288447"},
     {file = "lief-0.12.3-cp311-cp311-macosx_10_14_arm64.whl", hash = "sha256:b5c123cb99a7879d754c059e299198b34e7e30e3b64cf22e8962013db0099f47"},
     {file = "lief-0.12.3-cp311-cp311-macosx_10_14_x86_64.whl", hash = "sha256:8bc58fa26a830df6178e36f112cb2bbdd65deff593f066d2d51434ff78386ba5"},
+    {file = "lief-0.12.3-cp311-cp311-macosx_11_0_arm64.whl", hash = "sha256:74ac6143ac6ccd813c9b068d9c5f1f9d55c8813c8b407387eb57de01c3db2d74"},
     {file = "lief-0.12.3-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl", hash = "sha256:04eb6b70d646fb5bd6183575928ee23715550f161f2832cbcd8c6ff2071fb408"},
     {file = "lief-0.12.3-cp311-cp311-win32.whl", hash = "sha256:7e2d0a53c403769b04adcf8df92e83c5e25f9103a052aa7f17b0a9cf057735fb"},
     {file = "lief-0.12.3-cp311-cp311-win_amd64.whl", hash = "sha256:7f6395c12ee1bc4a5162f567cba96d0c72dfb660e7902e84d4f3029daf14fe33"},
@@ -1480,6 +1482,7 @@ files = [
     {file = "lief-0.12.3-cp38-cp38-win_amd64.whl", hash = "sha256:b00667257b43e93d94166c959055b6147d46d302598f3ee55c194b40414c89cc"},
     {file = "lief-0.12.3-cp39-cp39-macosx_10_14_arm64.whl", hash = "sha256:e6a1b5b389090d524621c2455795e1262f62dc9381bedd96f0cd72b878c4066d"},
     {file = "lief-0.12.3-cp39-cp39-macosx_10_14_x86_64.whl", hash = "sha256:ae773196df814202c0c51056163a1478941b299512b09660a3c37be3c7fac81e"},
+    {file = "lief-0.12.3-cp39-cp39-macosx_11_0_arm64.whl", hash = "sha256:66ddf88917ec7b00752687c476bb2771dc8ec19bd7e4c0dcff1f8ef774cad4e9"},
     {file = "lief-0.12.3-cp39-cp39-manylinux2014_aarch64.whl", hash = "sha256:4a47f410032c63ac3be051d963d0337d6b47f0e94bfe8e946ab4b6c428f4d0f8"},
     {file = "lief-0.12.3-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl", hash = "sha256:dbd11367c2259bd1131a6c8755dcde33314324de5ea029227bfbc7d3755871e6"},
     {file = "lief-0.12.3-cp39-cp39-win32.whl", hash = "sha256:2ce53e311918c3e5b54c815ef420a747208d2a88200c41cd476f3dd1eb876bcf"},
@@ -2352,7 +2355,7 @@ files = [
 cffi = ">=1.4.1"
 
 [package.extras]
-docs = ["sphinx (>=1.6.5)", "sphinx_rtd_theme"]
+docs = ["sphinx (>=1.6.5)", "sphinx-rtd-theme"]
 tests = ["hypothesis (>=3.27.0)", "pytest (>=3.2.1,!=3.3.0)"]
 
 [[package]]
@@ -3245,6 +3248,21 @@ files = [
 [package.dependencies]
 six = "*"
 
+[[package]]
+name = "wemake-python-styleguide"
+version = "0.0.1"
+description = "Opinionated styleguide that we use in wemake.services projects"
+category = "dev"
+optional = false
+python-versions = "*"
+files = [
+    {file = "wemake-python-styleguide-0.0.1.tar.gz", hash = "sha256:e1f47a2be6aa79ca8a1cfbbbffdd67bf4df32b76306f4c3dd2a620a2af78e671"},
+    {file = "wemake_python_styleguide-0.0.1-py2.py3-none-any.whl", hash = "sha256:505a19d82f9c4f450c6f06bb8c74d86c99cabcc4d5e6d8ea70e90b13b049f34f"},
+]
+
+[package.dependencies]
+flake8 = "*"
+
 [[package]]
 name = "wheel"
 version = "0.38.4"
@@ -3462,4 +3480,4 @@ testing = ["flake8 (<5)", "func-timeout", "jaraco.functools", "jaraco.itertools"
 [metadata]
 lock-version = "2.0"
 python-versions = ">=3.9.1,<3.10"
-content-hash = "02daca205796a0f29a0d9f50707544e6804f32027eba493cd2aa7f175a00dcea"
+content-hash = "c566a959134559e2fccf9b17fc820956207c9d36a58195f48990b986114d7108"
diff --git a/pyproject.toml b/pyproject.toml
index 2f40d58f56..99e03daea1 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -94,6 +94,7 @@ wheel = "*"
 enlighten = "*"  # cool terminal progress bars
 toml = "^0.10.2" # for parsing pyproject.toml
 pre-commit = "*"
+wemake-python-styleguide = "*"
 
 [tool.poetry.urls]
 "Bug Tracker" = "https://github.com/pypeclub/openpype/issues"

From a6f3bbe0f4f653048e11d7ecbb282d7c7583ef53 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 16:49:09 +0200
Subject: [PATCH 14/15] :wrench: update flake8 configuration

---
 setup.cfg | 144 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 140 insertions(+), 4 deletions(-)

diff --git a/setup.cfg b/setup.cfg
index 10cca3eb3f..838c6bd4c5 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -1,7 +1,6 @@
 [flake8]
-# ignore = D203
-ignore = BLK100, W504, W503
 max-line-length = 79
+strictness = short
 exclude =
   .git,
   __pycache__,
@@ -10,8 +9,145 @@ exclude =
   website,
   openpype/vendor,
   *deadline/repository/custom/plugins
-
 max-complexity = 30
+ignore =
+    # line break before binary operator
+    W503,
+    # line break occurred after a binary operator
+    W504,
+    # wemake-python-styleguide warnings
+    # See https://wemake-python-stylegui.de/en/latest/pages/usage/violations/index.html for doc
+    # Found incorrect module name pattern
+    WPS102,
+    # Found wrong variable name
+    WPS110,
+    # Found too short name
+    WPS111,
+    # Found upper-case constant in a class
+    WPS115,
+    # Found module with too many imports
+    WPS201,
+    # Found too many module members
+    WPS202,
+    # Found overused expression
+    WPS204,
+    # Found too many local variables
+    WPS210,
+    # Found too many arguments
+    WPS211,
+    # Found too many return statements
+    WPS212,
+    # Found too many expressions
+    WPS213,
+    # Found too many methods
+    WPS214,
+    # Found too many await expressions
+    WPS217,
+    # Found line with high Jones Complexity
+    WPS221,
+    # Found too many `elif` branches
+    WPS223,
+    # Found string constant over-use
+    WPS226,
+    # Found too long try body length
+    WPS229,
+    # Found too many public instance attributes
+    WPS230,
+    # Found function with too much cognitive complexity
+    WPS231,
+    # Found module cognitive complexity that is too high
+    WPS232,
+    # Found too many imported names from a module
+    WPS235,
+    # Found too many raises in a function
+    WPS238,
+    # Found too deep nesting
+    WPS220,
+    # Found `f` string
+    WPS305,
+    # Found incorrect multi-line parameters
+    WPS317,
+    # Found extra indentation
+    WPS318,
+    # Found bracket in wrong position
+    WPS319,
+    # Found percent string formatting
+    WPS323,
+    # Found implicit string concatenation
+    WPS326,
+    # Found variables that are only used for `return`
+    WPS331,
+    # Found explicit string concatenation
+    WPS336,
+    # Found multiline conditions
+    WPS337,
+    # Found incorrect order of methods in a class
+    WPS338,
+    # Found line starting with a dot
+    WPS348,
+    # Found multiline loop
+    WPS352,
+    # Found incorrect unpacking target
+    WPS414,
+    # Found wrong keyword
+    WPS420,
+    # Found wrong function
+    WPS421,
+    # Found statement that has no effect
+    WPS428,
+    # Found nested function
+    WPS430,
+    # Found magic number
+    WPS432,
+    # Found protected attribute usage
+    WPS437,
+    # Found block variables overlap
+    WPS440,
+    # Found an infinite while loop
+    WPS457,
+    # Found a getter without a return value
+    WPS463,
+    # Found negated condition
+    WPS504,
+    # flake8-quotes warnings
+    # Remove bad quotes
+    Q000,
+    # Remove bad quotes from multiline string
+    Q001,
+    # Darglint warnings
+    # Incorrect indentation
+    DAR003,
+    # Excess parameter(s) in Docstring
+    DAR102,
+    # Excess exception(s) in Raises section
+    DAR402,
+    # pydocstyle warnings
+    # Missing docstring in __init_
+    D107,
+    # White space formatting for doc strings
+    D2,
+    # First line should end with a period
+    D400,
+    # Others
+    # function name
+    N802,
+    # Found backslash that is used for line breaking
+    N400,
+    E501,
+    S105,
+    RST
+
+[isort]
+profile=wemake
+src_paths=isort,test
+# isort configuration:
+# https://github.com/timothycrosley/isort/wiki/isort-Settings
+include_trailing_comma = true
+use_parentheses = true
+# See https://github.com/timothycrosley/isort#multi-line-output-modes
+multi_line_output = 3
+# Is the same as 80 in flake8:
+line_length = 79
 
 [pylint.'MESSAGES CONTROL']
 disable = no-member
@@ -28,4 +164,4 @@ omit = /tests
 directory = ./coverage
 
 [tool:pytest]
-norecursedirs = repos/* openpype/modules/ftrack/*
\ No newline at end of file
+norecursedirs = repos/* openpype/modules/ftrack/*

From e1384a40b1b94a0f456d6cd4e01a63dd1e8a23e5 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@ynput.io>
Date: Thu, 4 May 2023 17:46:31 +0200
Subject: [PATCH 15/15] :heavy_plus_sign: add isort and fix python black check

---
 poetry.lock    | 2 +-
 pyproject.toml | 1 +
 setup.cfg      | 4 +++-
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/poetry.lock b/poetry.lock
index cbc4ac7b94..563f905fad 100644
--- a/poetry.lock
+++ b/poetry.lock
@@ -3480,4 +3480,4 @@ testing = ["flake8 (<5)", "func-timeout", "jaraco.functools", "jaraco.itertools"
 [metadata]
 lock-version = "2.0"
 python-versions = ">=3.9.1,<3.10"
-content-hash = "c566a959134559e2fccf9b17fc820956207c9d36a58195f48990b986114d7108"
+content-hash = "45e91b47f9e6697b0eb9fdbe76981f691d389ce74bc5a0e98d72e1109b39bc63"
diff --git a/pyproject.toml b/pyproject.toml
index 4bb80c060b..0d236aedc6 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -95,6 +95,7 @@ enlighten = "*"  # cool terminal progress bars
 toml = "^0.10.2" # for parsing pyproject.toml
 pre-commit = "*"
 wemake-python-styleguide = "*"
+isort="*"
 
 [tool.poetry.urls]
 "Bug Tracker" = "https://github.com/pypeclub/openpype/issues"
diff --git a/setup.cfg b/setup.cfg
index 838c6bd4c5..7863a74894 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -135,7 +135,9 @@ ignore =
     N400,
     E501,
     S105,
-    RST
+    RST,
+    # Black would make changes error
+    BLK100,
 
 [isort]
 profile=wemake