Skip to content

Use Git instead of go-git #7

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Feb 17, 2023
Merged

Use Git instead of go-git #7

merged 17 commits into from
Feb 17, 2023

Conversation

spenserblack
Copy link
Owner

@spenserblack spenserblack commented Feb 17, 2023

This switches out dependence on the go-git module with just using the installed git executable. This drastically reduces the number of dependencies, and hopefully improves compatibility (if git changes their plumbing, go-git might break, breaking this executable).

Changelog

Fixed

  • Renames getting marked as an add and delete
  • Missing support for file type changes

Because of an incompatible change in return types, the deprecated
functions can't act as wrappers around the new functions. Because of
this, a breaking change is unavoidable to the library.
@spenserblack spenserblack marked this pull request as ready for review February 17, 2023 20:55
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #7 (77c39f9) into main (bae878c) will increase coverage by 6.10%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   71.42%   77.53%   +6.10%     
==========================================
  Files           3        5       +2     
  Lines         112      138      +26     
==========================================
+ Hits           80      107      +27     
+ Misses         25       24       -1     
  Partials        7        7              
Impacted Files Coverage Δ
worktree.go 50.00% <50.00%> (ø)
repo.go 62.50% <62.50%> (ø)
commit.go 65.95% <68.00%> (+8.58%) ⬆️
status.go 83.01% <83.01%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@spenserblack spenserblack merged commit f68a84b into main Feb 17, 2023
@spenserblack spenserblack deleted the chore/remove-go-git branch February 17, 2023 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant