fix(fixtures): evaluate the rc of git-cliff correctly

[Cyclonit]

Description

Fixes the test fixtures for #1061.

Motivation and Context

#1061 changed the fixtures system to allow checking git-cliff's return code. This unintentionally broke all fixtures. The problem was caused by GitHub running all bash scripts with set -e by default, leading to any failing command failing the entire script. I circumvented this by appending the git-cliff command with || true, thereby breaking the subsequent checks.

This PR fixes the problem by adding set +e prior to running git-cliff, thus allowing evaluation of its rc as intended.

Additionally it fixes the fixture test-require-conventional-skipped. This fixture never worked right, but was deemed successful due to the bug.

How Has This Been Tested?

Running fixtures locally and in GitHub actions.

Screenshots / Logs (if applicable)

Types of Changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation (no code change)
• Refactor (refactoring production code)
• Other

Checklist:

• My code follows the code style of this project.
• I have updated the documentation accordingly.
• I have formatted the code with rustfmt.
• I checked the lints with clippy.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.47%. Comparing base (f302e43) to head (dad9759).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1104      +/-   ##
==========================================
+ Coverage   41.42%   41.47%   +0.06%     
==========================================
  Files          21       21              
  Lines        1811     1811              
==========================================
+ Hits          750      751       +1     
+ Misses       1061     1060       -1     

Flag	Coverage Δ
unit-tests	41.47% <ø> (+0.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[orhun]

Nice, thanks!