Chore: split circle ci jobs

pustovitDmytro · Apr 27, 2021 · 9f75a6b · 9f75a6b
1 parent 82f4156
commit 9f75a6b
Show file tree

Hide file tree

Showing 3 changed files with 46 additions and 26 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -66,22 +66,36 @@ jobs:
       - run: *install
       - run: *security
       - save_cache: *cache-modules
-  tests:
+  test-coverage:
     <<: *defaults
     steps:
       - checkout
       - restore_cache:
           <<: *cache-defaults
-      - run: *lint
-      - run: *debt
       - run: *coverage
       - store_test_results:
           path: reports/test-results
       - store_artifacts:
           path: reports
-      - store_artifacts:
-          path: tmp/tests
       - run: *coveralls
+  test-lint:
+    <<: *defaults
+    steps:
+      - checkout
+      - restore_cache:
+          <<: *cache-defaults
+      - run: *lint
+      - store_test_results:
+          path: reports/test-results
+  test-debt:
+    <<: *defaults
+    steps:
+      - checkout
+      - restore_cache:
+          <<: *cache-defaults
+      - run: *debt
+      - store_artifacts:
+          path: reports
   test-package:
     <<: *defaults
     steps:
@@ -103,26 +117,32 @@ jobs:
 
 workflows:
   version: 2
-  # main:
-  #   jobs:
-  #     - install
-  #     - tests:
-  #         requires:
-  #           - install
-  #     - test-package:
-  #         requires:
-  #           - install
-  #     - deploy:
-  #         requires:
-  #           - tests
-  #           - test-package
-  #         filters:
-  #           branches:
-  #               only: master
+  main:
+    jobs:
+      - install
+      - test-package:
+          requires:
+            - install
+      - test-lint:
+          requires:
+            - install
+      - test-debt:
+          requires:
+            - install
+      - test-coverage:
+          requires:
+            - install
+      - deploy:
+          requires:
+            - test-coverage
+            - test-package
+          filters:
+            branches:
+                only: master
   pr:
-    # filters:
-    #     branches:
-    #       only: /pull\/[0-9]+/
+    filters:
+        branches:
+          only: /pull\/[0-9]+/
     jobs:
       - danger-pr
   env-tests:

diff --git a/.dangerfile.js b/.dangerfile.js
@@ -9,7 +9,7 @@ const isOwner = danger.github.pr.user.login === danger.github.thisPR.owner;
 const modifiedList = danger.git.modified_files.join('\n\n- ');
 
 export default async function () {
-    message(`Changed Files in this PR: \n- ${modifiedList}`);
+    message(`Changed Files in this PR:\n\n- ${modifiedList}`);
 
     if (system.modified && !isOwner) {
         const files = system.getKeyedPaths().modified;

diff --git a/package.json b/package.json
@@ -7,7 +7,7 @@
     "prepare": "babel src --out-dir ./lib && husky install",
     "coveralls": "cat ./reports/coverage/lcov.info | coveralls",
     "test:security": "npm audit && npm run test:lock",
-    "test:lint": "eslint src",
+    "test:lint": "eslint src bin tests",
     "test:mocha": "mocha  --config .mocharc.json tests/**/*test.js",
     "test:package": "./bin/pack.sh && ENTRY=./tmp/package/lib npm run test:mocha",
     "test:coverage": "BUILD=0 MODE=test nyc mocha --config .mocharc.json tests/**/*test.js",