Merge branch '1-change_variable_behaviour' into 'master'

feat: Change boolean variable behaviour

Closes #1

See merge request !1

README.md

@@ -29,7 +29,7 @@ It uses the following variable:
 | `CYPRESS_IMAGE`       | The Docker image used to run Cypress (use [included images](https://github.com/cypress-io/cypress-docker-images/tree/master/included) only). | `cypress/included:4.6.0` |
 | `CYPRESS_PROJECT_DIR` | The Cypress project directory (containing `cypress.json`) | `.` |
 | `CYPRESS_EXTRA_ARGS`  | Cypress extra [run options](https://docs.cypress.io/guides/guides/command-line.html#cypress-run) (to select a different browser, configuration or spec files for e.g.) | _none_ |
-| `REVIEW_ENABLED`      | Set to enable Cypress tests on review environments (dynamic environments instantiated on development branches) | _none_ (disabled) |
+| `REVIEW_ENABLED`      | Set to `true` to enable Cypress tests on review environments (dynamic environments instantiated on development branches) | _none_ (disabled) |
 
 ### Unit tests report integration
 

templates/gitlab-ci-cypress.yml

@@ -200,7 +200,7 @@ stages:
     fi
   }
 
-  if [[ -z "$TEMPLATE_CHECK_UPDATE_DISABLED" ]]; then check_for_update cypress "1.2.0"; fi
+  if [[ "$TEMPLATE_CHECK_UPDATE_DISABLED" != "true" ]]; then check_for_update cypress "1.2.0"; fi
   unscope_variables
 
   # ENDSCRIPT
@@ -251,6 +251,6 @@ cypress:
     # on production and integration branch(es): auto (on_success)
     - if: '$CI_COMMIT_REF_NAME =~ $PROD_REF || $CI_COMMIT_REF_NAME =~ $INTEG_REF'
     # on other branch(es): manual & non-blocking if $REVIEW_ENABLED set
-    - if: $REVIEW_ENABLED
+    - if: '$REVIEW_ENABLED == "true"'
       when: manual
       allow_failure: true