#7836 Merge request shows 0 commits, if upstream has new commits
closed
General
2015-08-20
2015-02-18
No
A merge request works ok if the downstream (forked) repo has all of the upstream (parent) repo's commits. But if the upstream repo has new commits on it, then the merge request shows 0 commits. This is because MergeRequest._commits can't find the upstream HEAD in the forked repo, so the log() doesn't work. We should have a smarter technique for finding the list of added commits. We should also have better error handling and message to the user when it can't be found since that'll probably still happen sometimes.
I think git merge-base looks like a better command to use than log. We'd need to add one repo as a remote into the other. I think it'd be reasonable to add the upstream one to the forked one. I haven't looked into options for Mercurial.
Do we need to update merge request when there are new commits on downstream (source) branch?
Right now we save HEAD of the source branch as it was at the moment of merge request creation, and show only those commits, but I think it would be nice to update merge request to the latest commits in the source branch. And it might make things simpler for hg (don't sure about that 100%, but seems like it).
It also would affect [#7830]. We'll need to merge latest commits, instead of relying on saved
commit_id, which can be deleted, if we decide that we want this behavior. And change merge instructions also.What do you think?
Related
Tickets:
#7830I am not 100% sure. We have [#5993] about the issue too. My initial thoughts are that automatically updating is the most useful, although it potentially could go wrong in some cases (e.g. if you make your changes on a master branch of a fork, and then make some other changes again on the master branch, those really shouldn't be part of the merge request). It certainly would be good and fine if people always use branches for each merge request.
I kind of like the current behavior (noted in [#5993]) where you can edit and re-save the merge request to update it. Then it is an explicit action that someone needs to take if they have pushed further changes. This functionality is not very obvious now, though.
Related
Tickets: #5993
Yeah, I think current behavior makes sense if we can update merge request, but we should make it more obvious to the user that such possibility exists
Closed #739.
Allura:
ib/7836ForgeHg merge request: https://sourceforge.net/p/forgehg/code/merge-requests/4/
Last edit: Igor Bondarenko 2015-04-22