Misleading message in PR diff view "File was deleted in this version"
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.
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 ?
Updated by Olivier Renaud 9 months ago
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
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
Updated by Redmine Integration 9 months ago
- Status changed from New to Resolved
d193c5b3a65b by Daniel Dourvaris firstname.lastname@example.org on
default branch changed this issue.