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

[David Greene via cfe-dev]

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