Removed all conditional checks in travis

.travis.yml

@@ -2,50 +2,31 @@ dist: bionic
 
 language: node_js
 
-jobs:
-  fast_finish: true
-  include:
-    - name: "Docker publish"
-      node_js: '12.16.3'
-      if: NOT type = pull_request
-      env:
-        - DOCKER_PUBLISH="true"
-    - name: "CI"
-      node_js: '12.16.3'
-      env:
-        - DOCKER_PUBLISH="false"
-  allow_failures:
-    - name: "Docker publish"
+branches:
+  only:
+    - /.*/
+
+services:
+  - docker
 
 cache:
   directories:
     - node_modules
 
-services:
-  - docker
-
-install:
-  - if [[ ${DOCKER_PUBLISH} == 'true' ]]; then sudo bash ./scripts/docker/install-docker ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then npm ci ; fi
-
-before_script:
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then echo "Building commit range ${TRAVIS_COMMIT_RANGE}" ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then export MUTATION_FILES=$(git diff ${TRAVIS_COMMIT_RANGE:-origin/master} --name-only | grep -E 'src\/(.*).(js|ts|jsx|tsx)$' | paste -sd ",") ; fi
-
-script:
-  - if [[ ${DOCKER_PUBLISH} == 'true' ]]; then bash ./scripts/docker/build ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then npm run lint ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then npm run test:ci ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then docker build -t shlink-web-client:test . ; fi
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then npm run mutate:ci ; fi
-
-after_success:
-  - if [[ ${DOCKER_PUBLISH} == 'false' ]]; then node_modules/.bin/ocular coverage/clover.xml ; fi
-
+jobs:
+  fast_finish: true
+  include:
+    - name: "Docker publish"
+      if: NOT type = pull_request
+      install: sudo bash ./scripts/docker/install-docker
+      script: bash ./scripts/docker/build
+      # Overwrite all common steps that have to be different
+      before_script: echo "Before script"
+      after_success: echo "After success"
+    - name: "CI"
+      node_js: '12.16.3'
       # Before deploying, build dist file for current travis tag
-before_deploy:
-  - if [[ ! -z $TRAVIS_TAG && ${DOCKER_PUBLISH} == 'false' ]]; then npm run build ${TRAVIS_TAG#?} ; fi
-
+      before_deploy: npm run build ${TRAVIS_TAG#?}
       deploy:
         - provider: releases
           api_key:
@@ -53,6 +34,20 @@ deploy:
           file: "./dist/shlink-web-client_${TRAVIS_TAG#?}_dist.zip"
           skip_cleanup: true
           on:
-      all_branches: true
-      condition: ${DOCKER_PUBLISH} == 'false'
             tags: true
+
+install:
+  - npm ci
+
+before_script:
+  - echo "Building commit range ${TRAVIS_COMMIT_RANGE}"
+  - export MUTATION_FILES=$(git diff ${TRAVIS_COMMIT_RANGE:-origin/master} --name-only | grep -E 'src\/(.*).(js|ts|jsx|tsx)$' | paste -sd ",")
+
+script:
+  - npm run lint
+  - npm run test:ci
+  - docker build -t shlink-web-client:test .
+  - npm run mutate:ci
+
+after_success:
+  - node_modules/.bin/ocular coverage/clover.xml