refactor(lib): clean up some code

[DaniPopes]

Description

Some clean up opportunities I found while reading the code.

No functional changes intended.

Please let me know if breaking changes are a concern for you, I can revert the change to the public API in the core crate.

Motivation and Context

Refactor

How Has This Been Tested?

cargo test + cargo clippy

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.

[welcome]

Thanks for opening this pull request! Please check out our contributing guidelines! ⛰️

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 1.31579% with 75 lines in your changes missing coverage. Please review.

Project coverage is 37.04%. Comparing base (35b886e) to head (51f5b4d).
Report is 195 commits behind head on main.

Files with missing lines	Patch %	Lines
git-cliff/src/lib.rs	0.00%	55 Missing ⚠️
git-cliff-core/src/repo.rs	0.00%	18 Missing ⚠️
git-cliff-core/src/changelog.rs	33.34%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #709      +/-   ##
==========================================
+ Coverage   36.66%   37.04%   +0.38%     
==========================================
  Files          20       20              
  Lines        1544     1531      -13     
==========================================
+ Hits          566      567       +1     
+ Misses        978      964      -14     

Flag	Coverage Δ
unit-tests	37.04% <1.32%> (+0.38%)	⬆️

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]

Thanks for this! It mostly looks good, the behavior looks like the same. I have some concerns about lib.rs but will double check to make sure.

[DaniPopes]

@orhun should be good now 🙏

[welcome]

Congrats on merging your first pull request! ⛰️

[orhun]

Awesome work, thanks a lot!