Merge branch '73-commitlint' into 'master'

feat: force commitlint jobs to fail by default

Closes #73

See merge request !87

CHANGELOG.md

@@ -5,6 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [4.2.0] - 2024-05-20
+
+### Changed
+
+- Remove `allow_failure: true` from `commitlint` and `commitlint-hotfix` jobs.
+  This behaviour is no longer desirable as we've built our whole `release-it`
+  process around the `conventionalcommits` specification. It's now more
+  beneficial for these jobs to fail by default.
+
 ## [4.1.0] - 2024-05-16
 
 ### Added

commitlint.yml

@@ -52,9 +52,6 @@ commitlint:
       - if: $COMMITLINT_DISABLED || ($CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH) || ($CI_COMMIT_BRANCH =~ /^release-it--.*$/i)
         when: never
       - if: ($CI_COMMIT_BRANCH && $CI_COMMIT_BRANCH !~ /^hotfix-.*$/i) || $CI_MERGE_REQUEST_IID
-        # Initially this job is for information only and we don't want to force pipelines to fail with incorrect commit
-        # messages so we set allow_failure to true.
-        allow_failure: true
 
 commitlint-hotfix:
   extends: .commitlint-base
@@ -68,6 +65,3 @@ commitlint-hotfix:
       - if: $COMMITLINT_HOTFIX_DISABLED
         when: never
       - if: $CI_COMMIT_BRANCH =~ /^hotfix-.*$/i
-        # Initially this job is for information only and we don't want to force pipelines to fail with incorrect commit
-        # messages so we set allow_failure to true.
-        allow_failure: true