yp05327
yp05327
We can add a test for this new feature later. Maybe not in this PR, it is not easy to do this, as we need to make many changes in...
This PR will not fix merged PR, as it will be fixed in #25810
There's a problem if we do not use branch id to link the head branch and the pull request. If we merged a PR, deleted the branch(ID:1) of that PR...
> Any tests possible? I think an unit test is possible.
I'm coding the test, and notice that maybe we need to check the repo access permission here, but not sure about this now. Maybe user can get new branches notification...
Ready for review.
Found a new problem: If user A fork a repo which have a recently commit pushed by user B, user A will see the notification immediately, as `branch.pusher` will be...
> Pusher is a user which have registered in this Gitea instance which is different from committer. Git Committer maybe not a user of this instance. i.e. I pushed git...
> Is it ready? Yes.
Of course it is better to reuse the existing fixtures in some simple tests. At first, I did this. But I found that for a complex test, reusing them will...