[llvm-dev] [cfe-dev] Phabricator -> GitHub PRs?

David Greene via llvm-dev llvm-dev at lists.llvm.org
Thu Jan 23 08:37:28 PST 2020


Hubert Tong <hubert.reinterpretcast at gmail.com> writes:

>> I read this as the refresh being an entirely new GitHub PR.  Is that
>> right?  Normally I would expect the same PR to be used but the rebase
>> would cause a force-push of the branch which would update the PR with
>> the new commits but might lose comments.  It's that later part I'm
>> unsure about.  It would seem odd to me to open an entirely new PR due to
>> a rebase/update of commits to respond to review.
>>
> Use of force push damages the ability to retrieve context on older
> comments. I am not sure of the reason for the case I observed, but the
> context vanished within a week in one instance.

Does "vanish" mean it was completely gone, or just hidden in some way?

                      -David


More information about the llvm-dev mailing list