chore: refactor ci workflow so that lint depends on compile

spiltcoffee · Mar 9, 2019 · 4a1e379 · 4a1e379
1 parent b158774
commit 4a1e379
Showing 1 changed file with 20 additions and 16 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -30,7 +30,7 @@ jobs:
             - packages/@postdfm/dfm2ast/node_modules
           key: v1-dependencies-{{ checksum "yarn.lock" }}
 
-  lint:
+  compile:
     executor: node-executor
     steps:
       - checkout
@@ -39,9 +39,15 @@ jobs:
             - v1-dependencies-{{ checksum "yarn.lock" }}
             - v1-dependencies-
 
-      - run: yarn lint:check
+      - run: yarn compile
 
-  format:
+      - persist_to_workspace:
+          root: packages
+          paths:
+            - ./postdfm/dist
+            - ./@postdfm/dfm2ast/dist
+
+  lint:
     executor: node-executor
     steps:
       - checkout
@@ -50,9 +56,12 @@ jobs:
             - v1-dependencies-{{ checksum "yarn.lock" }}
             - v1-dependencies-
 
-      - run: yarn format:check
+      - attach_workspace:
+          at: ~/repo/packages
 
-  compile:
+      - run: yarn lint:check
+
+  format:
     executor: node-executor
     steps:
       - checkout
@@ -61,13 +70,8 @@ jobs:
             - v1-dependencies-{{ checksum "yarn.lock" }}
             - v1-dependencies-
 
-      - run: yarn compile
+      - run: yarn format:check
 
-      - persist_to_workspace:
-          root: packages
-          paths:
-            - ./postdfm/dist
-            - ./@postdfm/dfm2ast/dist
   test:
     executor: node-executor
     steps:
@@ -104,18 +108,18 @@ workflows:
   ci:
     jobs:
       - bootstrap
-      - lint:
+      - compile:
           requires:
             - bootstrap
+      - lint:
+          requires:
+            - compile
       - format:
           requires:
             - lint
-      - compile:
-          requires:
-            - format
       - test:
           requires:
-            - compile
+            - format
       - release:
           requires:
             - test