Skip to content

Conversation

@mtth
Copy link
Owner

@mtth mtth commented Apr 17, 2025

This is a prerequisite to improving the conflict/merge flow.

Also refactor to omit the gitpython dependency.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 92.79279% with 8 lines in your changes missing coverage. Please review.

Project coverage is 92.07%. Comparing base (f8d0b32) to head (a20cc4d).

Files with missing lines Patch % Lines
src/git_draft/drafter.py 86.36% 4 Missing and 2 partials ⚠️
src/git_draft/git.py 98.11% 1 Missing ⚠️
src/git_draft/toolbox.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
+ Coverage   91.94%   92.07%   +0.13%     
==========================================
  Files           9       10       +1     
  Lines         608      656      +48     
  Branches       43       46       +3     
==========================================
+ Hits          559      604      +45     
- Misses         32       33       +1     
- Partials       17       19       +2     

☔ 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.

@mtth mtth changed the title refactor: omit gitpython dependency feat: require sync Apr 17, 2025
@github-actions github-actions bot added feat and removed refactor labels Apr 17, 2025
@mtth mtth merged commit 5a46298 into main Apr 17, 2025
4 checks passed
@mtth mtth deleted the mtth/uyAgn1vT branch April 17, 2025 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants