When a reviewer comments a code line in a pull request and the author of the pull requests updates the same code line afterwards, in the pull request activity timeline it shows the updated code part instead of the code lines they were on time of the comment.
When using pull-requests as review feature this is very ugly because the comment does not match to the code.
Steps:
- Developer A creates a pull-request
- Developer B (reviewer) opens the pull request and comments a line in the diff window.
- Developer B does NOT mark it as "Needs work" (because he is maybe not yet finished)
- Developer A commits and pushes a new commit which changes the commented line again.
- When Developer A or B looks at the activity tab in the pull request the commented code line is from step 4 instead of the code line from step 1