Fix build runing twice on PRs + lint fix

GuySerfaty · GuySerfaty · commit df358130b4d0 · 2022-03-08T12:06:30.000+02:00
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -1,5 +1,14 @@
 name: build
-on: ["push", "pull_request"]
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    types: [opened, synchronize, reopened, edited]
+    branches:
+      - main
+      - "project/**"
+
 jobs:
   build:
     name: Build
diff --git a/packages/core/src/extensions/Blocks/OrderedListPlugin.ts b/packages/core/src/extensions/Blocks/OrderedListPlugin.ts
@@ -1,35 +1,42 @@
-import {Plugin, PluginKey} from "prosemirror-state";
+import { Plugin, PluginKey } from "prosemirror-state";
 
 const PLUGIN_KEY = new PluginKey(`ordered-list`);
 
 export const OrderedListPlugin = () => {
   return new Plugin({
     key: PLUGIN_KEY,
     appendTransaction: (_transactions, _oldState, newState) => {
-      const newTr = newState.tr
-      let modified = false
-      let count = 1
-      let skip = 0
+      const newTr = newState.tr;
+      let modified = false;
+      let count = 1;
+      let skip = 0;
       newState.doc.descendants((node, pos) => {
-        if (node.type.name === "tcblock" && !node.attrs.listType) count = 1
-        if (skip == 0 && node.type.name === "tcblock" && node.attrs.listType === "oli") {
-          skip = node.content.childCount
+        if (node.type.name === "tcblock" && !node.attrs.listType) count = 1;
+        if (
+          skip === 0 &&
+          node.type.name === "tcblock" &&
+          node.attrs.listType === "oli"
+        ) {
+          skip = node.content.childCount;
           // This assumes that the content node is always the first child of the oli block,
           // as the content model grows this assumption may need to change
           if (node.content.child(0).attrs.position !== `${count}.`) {
             // TODO: @DAlperin currently sub-items just continue from the order of the parent,
             //  sub-items should be ordered separately with letters or roman numerals or some such
-            newTr.setNodeMarkup(pos+1, undefined, {...node.attrs, position: `${count}.`})
-            modified = true
+            newTr.setNodeMarkup(pos + 1, undefined, {
+              ...node.attrs,
+              position: `${count}.`,
+            });
+            modified = true;
           }
 
-          count++
-        } else if (skip > 0) skip--
-      })
-      if (modified){
-        return newTr
+          count++;
+        } else if (skip > 0) skip--;
+      });
+      if (modified) {
+        return newTr;
       }
-      return null
-    }
-  })
-}
+      return null;
+    },
+  });
+};
