Skip to content

Conversation

@Chi-Iroh
Copy link

@Chi-Iroh Chi-Iroh commented Jul 8, 2025

#34686
I accidentally deleted the PR fork (I had many ones), so afaik I'm forced to create another PR.
Sorry for the inconvenience.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 8, 2025
@github-actions github-actions bot added modifies/translation modifies/go Pull requests that update Go code labels Jul 8, 2025
@github-actions github-actions bot added modifies/templates This PR modifies the template files modifies/frontend labels Jul 8, 2025
@Taldariner
Copy link

Hello!
Is there any updates on this PR?
Would be very grateful if this feature were added.

@Chi-Iroh
Copy link
Author

Chi-Iroh commented Oct 5, 2025

Hi @Taldariner !

My changes weren't approved as maintainers had to study some technical details, see here: #34686 (comment)

And it's my first contribution to this project, so it unfortunately slows the process down.

@Taldariner
Copy link

Taldariner commented Oct 5, 2025

Hi, @Chi-Iroh !

I read that conversation, but looks like after PR recreation process stubbed a little bit.

But anyway, I got it, hope it will be merged soon, and thank you for that work!

@lunny
Copy link
Member

lunny commented Oct 19, 2025

Could you post a screenshot and please have a test for the rename behavior.

@wxiaoguang
Copy link
Contributor

I can see there are still some problems in code, for example: we can't depend on the unstable ErrUnexpectedEOF behavior (there are other cases causing such error, and then the data would be wrong); the query parameter needs a complete backend implementation because users need to navigate between pages when the commits number is large; it needs some tests to cover the new behavior of CommitsByFileAndRange

It needs to be refactored & cleaned up first

@Chi-Iroh
Copy link
Author

Chi-Iroh commented Oct 24, 2025

Could you post a screenshot and please have a test for the rename behavior.
@lunny

image Comes from my very first message in #34686

@Chi-Iroh
Copy link
Author

@wxiaoguang I managed to change the format string in git log --follow to have a newline at the end, so I don't rely on ErrUnexpectedEOF anymore.

@Chi-Iroh
Copy link
Author

I'm facing an issue regarding my commits : as many commits were pushed to Gitea upstream since I made my fork, I have some conflicts.
Locally, I merged my branch with the upstream main to solve conflicts, but I have more than a thousand changes due to the commits on the upstream.
To keep the PR clean, what would you like me to do ? A commit containing solely the changes to fix the conflicts ? If so, my branch won't build as it will lack some recent changes.
Sorry for the inconvenience, I'm not sure how to proceed.

@lunny
Copy link
Member

lunny commented Oct 24, 2025

You can use Gitea’s main branch as the base branch when creating your pull request. After it’s merged, simply merge the main branch back into your private branch to keep it up to date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/frontend modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants