Feature #5615
open
Misleading message in PR diff view "File was deleted in this version"
Added by Olivier Renaud over 4 years ago.
Updated over 4 years ago.
Description
I have a PR with an inline code comment. I updated the PR with a new commit that reverts the code that the comment was attached to. Now, the PR page correctly keeps the inline comment in a dedicated block at the end of the diffs. My problem is that the message that appears in this block is misleading.
It says that the file was deleted, but it's not the case: there is just no change in this file anymore, in the complete diff.
Files
Hi Oliver,
This is somehow expected. Our system tracks changes of pull-requests, and there were actions attached to the mentioned file, that was now deleted in the next iteration of a pull-request. This is why we display this information. If you would go back to the previous version we'd still show that file.
Would it help if we work on the message and explain it more why we mark it as deleted in this version ?
I can see how technically a version of a PR "deleted" a file compared to a previous version. But the word "delete" really suggests that my PR includes the deletion of the file (as in hg remove or git rm).
I can suggest:
removed in this version -> no changes in this version
File was deleted in this version -> File does not contain changes anymore in this version
- Assignee set to Daniel D
- Target version set to v4.19
Thanks for the suggestion, you're right "deleted" really is too strong, and indicates permanent action.
We'll discuss this internally, and adjust the names for that case operation.
This is our current proposal, let us know what do you think?
I'm fine with this new message, yes. It's certainly less confusing than the current one.
Thanks.
- Status changed from New to Resolved
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by