-------- Weitergeleitete Nachricht -------- Betreff: Re: Github loses comments Datum: Tue, 07 Jul 2015 00:29:38 -0700 Von: Ivan Žužak support@github.com An: Thomas Martitz kugel@rockbox.org
Hi Thomas,
Thanks for sharing those details about your workflow.
If you create a comment on a commit which is a part of a pull request, and later on blow away that commit from the pull request with a rebase + force-push combo -- then that commit is no longer a part of the pull request so the comments are no longer shown inline on the pull request page.
However, if you create comments on the pull request's diff itself -- then those comments are tied to the pull request. After you update a pull request (even with a rebase), those comments will still be around on the pull request page, but will be shown as "user commented on an outdated diff a day ago".
So, my suggestion would be to comment on the pull request's diff if you're using a rebase-based workflow. We understand that commenting on specific commits is helpful in some situations, and we're looking into ways to keep comments around on pull request pages even after you rebase away the commits they were created on. I can't make any promises about this, but it is on our radar.
Let me know if I misunderstood anything, if you have any questions or if you want to share more details so that I can pass the feedback to the team.
Cheers, Ivan
Hello,
when a pull request is updated though a history rewrite (e.g. rebase, amend, commits reodered) to make the history accepatable for the main repository, then github sometimes loses comments associated with the pull request. This is annoying as it really makes reviewing *more* difficult. But only ever adding fixup commits is not acceptable either as it would add unwanted commits in the main repository (for example, if they break the build of a project it makes bisecting harder than necessary, therefore some commits are not acceptable for merging into the master branch).
Can this be fixed? I do notice that github some other times shows comments through "X commented on an outdated diff". This seems a lot better than deleting valuable comments altogether.
Best regards