Skip to content

refactor(BaseCommitizen): remove unused process_commit #1468

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: refactors
Choose a base branch
from

Conversation

bearomorphism
Copy link
Contributor

@bearomorphism bearomorphism commented May 30, 2025

Description

Discussed with @Lee-W offline. process_commit is introduced in #179 and nobody is using it(?

Checklist

Copy link

codecov bot commented May 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.81%. Comparing base (59fd3f5) to head (8540e1f).

Additional details and impacted files
@@              Coverage Diff              @@
##           refactors    #1468      +/-   ##
=============================================
- Coverage      97.82%   97.81%   -0.01%     
=============================================
  Files             57       57              
  Lines           2619     2612       -7     
=============================================
- Hits            2562     2555       -7     
  Misses            57       57              
Flag Coverage Δ
unittests 97.81% <ø> (-0.01%) ⬇️

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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants