diff --git a/.eslintrc.js b/.eslintrc.js index 8f5aedb..259de13 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -2,7 +2,7 @@ module.exports = { parser: '@typescript-eslint/parser', parserOptions: { project: 'tsconfig.json', - tsconfigRootDir : __dirname, + tsconfigRootDir: __dirname, sourceType: 'module', }, plugins: ['@typescript-eslint/eslint-plugin'], diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 4f5144b..465af43 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -12,7 +12,7 @@ test: stage: test image: docker/compose:latest variables: - DOCKER_TLS_CERTDIR: "" + DOCKER_TLS_CERTDIR: '' services: - docker:dind script: @@ -22,7 +22,7 @@ test: - docker exec -t v3-configuration-api sh -c "npm run test:integration:ci" coverage: /All files[^|]*\|[^|]*\s+([\d\.]+)/ rules: - - if: '$CI_MERGE_REQUEST_TARGET_BRANCH_NAME == $CI_DEFAULT_BRANCH || $CI_COMMIT_MESSAGE =~ /--check/ || $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH' + - if: '$CI_MERGE_REQUEST_TARGET_BRANCH_NAME == $CI_DEFAULT_BRANCH || $CI_COMMIT_MESSAGE =~ /--check/ || $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH' when: always ############### @@ -33,7 +33,7 @@ build: stage: build image: docker:20.10.22 variables: - DOCKER_TLS_CERTDIR: "" + DOCKER_TLS_CERTDIR: '' services: - docker:dind before_script: diff --git a/.prettierignore b/.prettierignore new file mode 100644 index 0000000..192c132 --- /dev/null +++ b/.prettierignore @@ -0,0 +1,6 @@ +node_modules +*.log* +.env +dist +coverage +.prettierrc.json diff --git a/.prettierrc b/.prettierrc.json similarity index 100% rename from .prettierrc rename to .prettierrc.json diff --git a/README.md b/README.md index 8d2e23d..24078bb 100644 --- a/README.md +++ b/README.md @@ -127,7 +127,7 @@ As mentionned earlier, RabbitMQ messages are sent after these events : Various messages are also sent for logging purpose. -## Tests +## Tests / ESLint / Prettier Tests are run outside the container for ease of use (switching between different environments inside containers using prisma is complicated and error prone). The integration tests use a dedicated database (see _db-test_ section of _docker-compose.yml_). @@ -144,6 +144,12 @@ npm run test:integration # coverage npm run test:cov + +# ESLint +npm run lint + +# Prettier +npm run pretty ``` ## License diff --git a/ci/Dockerfile b/ci/Dockerfile index 4b2a20c..26b388a 100644 --- a/ci/Dockerfile +++ b/ci/Dockerfile @@ -25,5 +25,9 @@ RUN npm run build # Run unit tests RUN npm run test:unit:ci +# ESLint / Prettier +RUN npm run lint:check +RUN npm run pretty:check + # Start the server CMD [ "node", "dist/main.js" ] diff --git a/package.json b/package.json index 76482e1..9516c57 100644 --- a/package.json +++ b/package.json @@ -14,6 +14,9 @@ "start:debug": "nest start --debug --watch", "start:prod": "node dist/main", "lint": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix", + "lint:check": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix-dry-run --ignore-path .gitignore", + "pretty:check": "./node_modules/.bin/prettier --check .", + "pretty": "./node_modules/.bin/prettier --write .", "test": "npm run migrate:test && dotenv -e .env.test jest", "test:unit": "jest --testPathPattern 'tests/unit/' --verbose", "test:unit:ci": "jest --testPathPattern 'tests/unit/' --coverage",