Pull request comment disappearing

Luis Tacón August 22, 2017

After a pull request is updated if a line contains a comment and that line is changed the comment disappears. 

Is that the normal behaviour?

1 answer

0 votes
Ana Retamal
Atlassian Team
Atlassian Team members are employees working across the company in a wide variety of roles.
August 23, 2017

Hi Luis, updates to a file in a pull request can cause comments to disappear. We attempt to put the comments back on the correct line however this is not always possible (for example when the line is removed from the file).

I don't know the particulars about your case, so I'm not able to tell  whether or not your comment should have been removed or not.

The Bitbucket team wrote a detailed description about this behaviour at Building inline comments for pull request and commits (see the "High Plains Drifter" section). This might help you understand how this issue is approached.

Lastly, are you using Bitbucket Cloud or Server? For Server, we have a feature request at BSERV-7534 You can vote for it or add your comments.

Hope this helps!

Ana

Luis Tacón August 23, 2017

Hello Ana,

 

Thanks for the response. 

 

It's not the case. We are using at this moment Bitbucket Cloud for the first time and trying it. 

 

In this particular case, we made a commit modifying a line with a comment and it disappear. This commit the only thing that does is renaming a variable in the line so it's very strange.

 

Yes we read the documentation about the "Hight Plains Drifter" and we thought the comment wasn't going to be removed. 

jredmond
Atlassian Team
Atlassian Team members are employees working across the company in a wide variety of roles.
August 23, 2017

Generally, if the line changes, it's in response to the comment(s), so the PR diff view hides comments that are possibly obsolete.

Look for a number next to the "Side-by-side diff" button on a file's header, like the 2 in this screenshot; if you click that, then you should be able to see the hidden comments on that file.Screen Shot 2017-08-23 at 11.01.14 AM.png

Luis Tacón August 23, 2017

Ah ok. That's the normal behaviour, the comment disappearing after a change on the line.

 

We though is a little hard to review the changes after the comment is gone. We need to open the box with the number "2" (on your example) a view what comment were made and in what lines to make sure the changes are fixed. Maybe we are using it wrong.

Thanks!

Luis Tacón August 25, 2017

Do we have a way to visualize comments next to the last version of the code?

Like # people like this
Paul Gates May 29, 2019

@jredmond What about the case where a side-by-side diff is not allowed?

  1.  A new file is created
  2. Comments are added for a change
  3. Changes are made and added to the pull request (this step may be repeated)

In this case, it seems that the comments are hidden as obsolete in the Unified View, but since the file is considered new, the side-by-side diff is not allowed.

Darius V July 1, 2019

Could you make a way to see pull request diff at the state when was made the comment?  

 

Now pull request shows diff from latest code. In other words - diff from specific commit and code where we want to merge to.

 

Also the code where we want to merge to might change. So migth be not fully solvable. But still better.

And for that case maybe just add screenshots of the code and comments.

abhishekrathore03 May 18, 2020

@jredmond , there is a button e.g. 17 prior comments 

image.png

But clicking on it doesn't do anything for me?

It's a normal Pull request created on https://bitbucket.org/ within a private repository

Jeff_Swartz April 8, 2022

PR comments do disappear in Bitbucket cloud when a file with PR comments has its filename renamed.

Like Nikita likes this
Jeff_Swartz April 8, 2022

When a file is renamed in the PR, the PR displays the contents of the file as all new even though nothing may have been changed at all in the file contents.

Like Nikita likes this
Nikita September 7, 2022

Hi, Atlassian Team!

When a file is renamed in the PR, the PR displays the contents of the file as all new even though nothing may have been changed at all in the file contents.

This creates a lot of problems for code review.

Do you consider this acceptable behavior? Are there any moves in this direction?

Suggest an answer

Log in or Sign up to answer
TAGS
AUG Leaders

Atlassian Community Events