Skip to content

Conversation

fcharlie
Copy link
Contributor

Recreate for #109

@fcharlie fcharlie requested a review from a team as a code owner March 13, 2025 05:16
@JamesB192
Copy link

Seems to work with a modest repo.

berg2 $ ~/go/bin/git-sizer
Processing blobs: 101555                        
Processing trees: 69385                        
Processing commits: 35267                        
Matching commits to trees: 35267                        
Processing annotated tags: 0                        
Processing references: 8                        
No problems above the current threshold were found

Copy link
Contributor

@bk2204 bk2204 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This mostly looks fine to me, but I see some changes I think are extraneous here that either should be explained or backed out. With those fixed, I think this should be fine to merge.

@bk2204
Copy link
Contributor

bk2204 commented Aug 20, 2025

By the way, my apologies for the long delay in getting to this. I'll try to see to it that we get to PRs on this repo in a more timely manner.

bk2204
bk2204 previously approved these changes Aug 25, 2025
@bk2204 bk2204 merged commit ea9bd12 into github:master Sep 3, 2025
4 checks passed
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.

3 participants