ci: add test and lint jobs to ci

Author: SethFalco

.editorconfig

@@ -10,3 +10,6 @@ insert_final_newline = true
 charset = utf-8
 indent_style = space
 indent_size = 4
+
+[*.{yml,yaml}]
+indent_size = 2

.gitlab-ci.yml

@@ -1,5 +1,20 @@
+before_script:
+  - npm ci --ignore-scripts --force
+
+build:
+  stage: build
+  script: npm run build
+  only:
+    - merge_requests
+
+lint:
+  stage: test
+  script: npm run lint
+  only:
+    - merge_requests
+
 test:
   stage: test
-  script: npm ci --ignore-scripts --force && npm run test
+  script: npm run test
   only:
     - merge_requests

dist/squire-raw.js

@@ -338,11 +338,13 @@
       if (!child || isLeaf(child)) {
         if (startOffset) {
           child = startContainer.childNodes[startOffset - 1];
-          let prev = child.previousSibling;
-          while (child instanceof Text && !child.length && prev && prev instanceof Text) {
-            child.remove();
-            child = prev;
-            continue;
+          const prev = child.previousSibling;
+          if (prev && prev instanceof Text) {
+            while (child instanceof Text && !child.length) {
+              child.remove();
+              child = prev;
+              continue;
+            }
           }
           if (child instanceof Text) {
             startContainer = child;

dist/squire-raw.mjs

@@ -340,11 +340,13 @@ var moveRangeBoundariesDownTree = (range) => {
     if (!child || isLeaf(child)) {
       if (startOffset) {
         child = startContainer.childNodes[startOffset - 1];
-        let prev = child.previousSibling;
-        while (child instanceof Text && !child.length && prev && prev instanceof Text) {
-          child.remove();
-          child = prev;
-          continue;
+        const prev = child.previousSibling;
+        if (prev && prev instanceof Text) {
+          while (child instanceof Text && !child.length) {
+            child.remove();
+            child = prev;
+            continue;
+          }
         }
         if (child instanceof Text) {
           startContainer = child;

dist/squire.js

@@ -60,7 +60,7 @@ const fixContainer = (
     root: Element | DocumentFragment,
 ): Node => {
     let wrapper: HTMLElement | null = null;
-    [...container.childNodes].forEach(child => {
+    [...container.childNodes].forEach((child) => {
         const isBR = child.nodeName === 'BR';
         if (!isBR && isInline(child)) {
             if (!wrapper) {

dist/squire.js.map

@@ -48,19 +48,17 @@ const moveRangeBoundariesDownTree = (range: Range): void => {
         if (!child || isLeaf(child)) {
             if (startOffset) {
                 child = startContainer.childNodes[startOffset - 1];
-                let prev = child.previousSibling;
+                const prev = child.previousSibling;
                 // If we have an empty text node next to another text node,
                 // just skip and remove it.
-                while (
-                    child instanceof Text &&
-                    !child.length &&
-                    prev &&
-                    prev instanceof Text
-                ) {
-                    child.remove();
-                    child = prev;
-                    continue;
+                if (prev && prev instanceof Text) {
+                    while (child instanceof Text && !child.length) {
+                        child.remove();
+                        child = prev;
+                        continue;
+                    }
                 }
+
                 if (child instanceof Text) {
                     startContainer = child;
                     startOffset = child.data.length;