ci(pre-commit): add pre-commit ci badge (#664)

* ci(pre-commit): add pre-commit ci badge

* ci(pre-commit ci): add eslint extra dependencies

* test(server test): remove to short timeout from rest tests

* ci(pre-commit ci): add eslint as dependenency

* ci(circle ci): remove linting from circle ci since its moved to pre-commit ci

.circleci/config.yml

@@ -9,7 +9,6 @@ workflows:
   test-build:
     jobs:
       - test-server
-      - lint
       - get-version:
           filters: &filters-build
             tags:
@@ -20,8 +19,6 @@ workflows:
 
       # - build-viewer:
       #     # built the npm package
-      #     requires:
-      #       - lint
       #     filters: &filters-build
       #       tags:
       #         only: /^[0-9]+\.[0-9]+\.[0-9]+$/
@@ -38,7 +35,6 @@ workflows:
           context: &docker-hub-context
             - docker-hub
           requires:
-            - lint
             - test-server
             - get-version
 
@@ -46,14 +42,12 @@ workflows:
           context: *docker-hub-context
           filters: *filters-build
           requires:
-            - lint
             - get-version
 
       - docker-build-and-publish-webhooks:
           context: *docker-hub-context
           filters: *filters-build
           requires:
-            - lint
             - get-version
             - test-server
 
@@ -61,7 +55,6 @@ workflows:
           context: *docker-hub-context
           filters: *filters-build
           requires:
-            - lint
             - get-version
             - test-server
 
@@ -69,7 +62,6 @@ workflows:
           context: *docker-hub-context
           filters: *filters-build
           requires:
-            - lint
             - get-version
             - test-server
 
@@ -77,14 +69,12 @@ workflows:
           context: *docker-hub-context
           filters: *filters-build
           requires:
-            - lint
             - get-version
             - test-server
 
       - publish-helm-chart:
           filters: *filters-build
           requires:
-            - lint
             - test-server
             - get-version
             - docker-build-and-publish-server
@@ -96,8 +86,6 @@ workflows:
 
       - publish-npm:
           filters: *filters-build
-          requires:
-            - lint
 
 jobs:
   lint:

.pre-commit-config.yaml

@@ -5,6 +5,11 @@ repos:
       - id: eslint
         # files: \.[jt]sx?$ # *.js, *.jsx, *.ts and *.tsx
         # types: [file]
+        additional_dependencies:
+          - eslint@8.11.0
+          - eslint-config-prettier@8.5.0
+          - eslint-plugin-vue@8.5.0
+
   - repo: https://github.com/pre-commit/mirrors-prettier
     rev: 'v2.5.1' # Use the sha / tag you want to point at
     hooks:

packages/server/modules/core/tests/rest.spec.js

@@ -226,7 +226,7 @@ describe('Upload/Download Routes @api-rest', () => {
             }
           })
       })
-  }).timeout(5000)
+  })
 
   it('Should properly download an object, with all its children, into a text/plain response', (done) => {
     request(app)

packages/server/readme.md

@@ -5,6 +5,7 @@
 #### Status
 
 [![Speckle-Next](https://circleci.com/gh/specklesystems/speckle-server.svg?style=svg&circle-token=76eabd350ea243575cbb258b746ed3f471f7ac29)](https://github.com/Speckle-Next/SpeckleServer/) [![codecov](https://codecov.io/gh/specklesystems/speckle-server/branch/master/graph/badge.svg)](https://codecov.io/gh/specklesystems/speckle-server)
+[![pre-commit.ci status](https://results.pre-commit.ci/badge/github/specklesystems/speckle-server/main.svg)](https://results.pre-commit.ci/latest/github/specklesystems/speckle-server/main)
 
 ## Disclaimer