diff options
| author | Carlos Garnacho <carlosg@gnome.org> | 2022-02-19 10:50:15 +0100 |
|---|---|---|
| committer | Carlos Garnacho <carlosg@gnome.org> | 2022-03-08 11:31:15 +0100 |
| commit | 56cf628e473a4b4c125dd7750be2b0572779e1e8 (patch) | |
| tree | 4a23839fffef33815a2282563e2a90527e432ec0 /.gitlab-ci.yml | |
| parent | b5d4fae5ef123907fc4cfa98406224fd6c408539 (diff) | |
| download | tracker-56cf628e473a4b4c125dd7750be2b0572779e1e8.tar.gz | |
ci: Run check-commits job in normal pipelines too
Make it non-failable there though, so contributors can check the
errors before filing a merge request, but still being able to run
the rest of the pipeline.
Diffstat (limited to '.gitlab-ci.yml')
| -rw-r--r-- | .gitlab-ci.yml | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 8138a8279..0db9745f6 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -50,12 +50,16 @@ check-commit-log: GIT_DEPTH: "100" stage: review script: - - if [[ x"$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" != "x" ]] ; + - ci-fairy check-commits --junit-xml=check-junit-report.xml && exit 0; + if [[ -z "$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" ]] ; then - ci-fairy check-commits --junit-xml=check-junit-report.xml ; - else - echo "Not a merge request" ; - fi + echo "'ci-fairy check-commits' found commit messages with unexpected format, see https://wiki.gnome.org/Git/CommitMessages for the recommended guidelines. Please fix those before filing a merge request, this will be handled as a pipeline failure there." ; + exit 255 ; + fi ; + exit 1 + allow_failure: + exit_codes: + - 255 <<: *check check-merge-request: |