Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix GetCommitBranchStart bug #33298

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

Fix GetCommitBranchStart bug #33298

wants to merge 1 commit into from
+49 −30

Conversation

Zettat123
Copy link
Contributor

@Zettat123 Zettat123 commented Jan 16, 2025 •
edited
Loading

Fix #33265
Fix #33370

This PR also fixes some bugs in TestGitGeneral.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 16, 2025
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 16, 2025
@github-actions github-actions bot added the modifies/go Pull requests that update Go code label Jan 16, 2025
@pull-request-size pull-request-size bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 16, 2025
@pull-request-size pull-request-size bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 17, 2025
@lunny lunny added the type/bug label Jan 18, 2025
@Zettat123 Zettat123 requested a review from lunny January 24, 2025 02:44
@Zettat123 Zettat123 mentioned this pull request Jan 24, 2025
Closed
@Zettat123 Zettat123 mentioned this pull request Jan 26, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wxiaoguang wxiaoguang wxiaoguang left review comments

@lunny lunny Awaiting requested review from lunny

Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/go Pull requests that update Go code size/M Denotes a PR that changes 30-99 lines, ignoring generated files. type/bug
Projects
None yet
Milestone
No milestone
4 participants
@Zettat123 @lunny @wxiaoguang @GiteaBot