Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR shows completely different changes in GitHub view vs Pull Request view #3260

Closed
RikkiGibson opened this issue Jan 18, 2022 · 8 comments
Closed
Assignees

Comments

@RikkiGibson
Copy link
Member

RikkiGibson commented Jan 18, 2022

  • Extension version (web): v0.34.3
  • VSCode Version (web): 1.63.2
  • Browser: Microsoft Edge Version 97.0.1072.62

Steps to Reproduce:

  1. Open https://github.dev/dotnet/roslyn/pull/57667
  2. Observe changes in CSharpCompilation.cs in "GitHub Pull Request" tab
  3. Open the same PR in the "GitHub" tab and find the changes in CSharpCompilation.cs

Expected behavior: equivalent file changes shown in both places
Actual behavior: different file changes are shown in each place

Note that the changes in "GitHub" are correct while the changes in "GitHub Pull Request" are incorrect.

GitHub view:
image

GitHub Pull Request view:
image

@alexr00 alexr00 self-assigned this Jan 25, 2022
@alexr00 alexr00 added the bug Issue identified by VS Code Team member as probable bug label Jan 25, 2022
@alexr00 alexr00 added this to the February 2022 milestone Jan 25, 2022
@alexr00
Copy link
Member

alexr00 commented Jan 25, 2022

@RikkiGibson Does this happen every time you open that PR? I'm not able to repro the issue.

If it does happen every time, would you be willing to try the pre-release version of the extension and see if it also reproduces there?

@RikkiGibson
Copy link
Member Author

I looked again and the "GitHub Pull Request" view is still wrong, but wrong in a new way :)

image

Switching to pre-release actually fixes the issue. Thanks for the tip!

I wonder if a specific issue was fixed between stable and pre-release? Did you use the version from the issue description when attempting to repro?

@alexr00
Copy link
Member

alexr00 commented Jan 26, 2022

I tried both stable and pre-release when trying to repro, but no luck on either. There are some significant refactors to improve performance in pre-release, so something could have fixed it.

@alexr00
Copy link
Member

alexr00 commented Feb 22, 2022

@RikkiGibson do you still see this? I haven't seen the issue through my daily use yet.

@alexr00 alexr00 modified the milestones: February 2022, March 2022 Feb 22, 2022
@RikkiGibson
Copy link
Member Author

I think dotnet/roslyn#57667 still shows some bad behavior in my Edge, but other pull requests seem to work fine. I wonder if some local cache got into a bad state specifically for that pull request.

@alexr00
Copy link
Member

alexr00 commented Feb 23, 2022

I wonder if some local cache got into a bad state specifically for that pull request.

That could be it. Could you try going to https://vscode.dev/github/dotnet/roslyn/pull/57667 and seeing if it repros there?

@RikkiGibson
Copy link
Member Author

Fascinating. Right now I get the gray bars indicating I can leave a comment, but no actual content diffs in github.dev:

image

In vscode.dev/github I get the proper content diffs:

image

@alexr00
Copy link
Member

alexr00 commented Feb 23, 2022

The no diffs is caused by microsoft/vscode-remote-repositories-github#121 We're still trying to get to the bottom of that one.

I think it's safe to say that the original issue was caused by caching though if you can't repro it on vscode.dev. If you see the incorrect diffs (not the no diffs) again, lets reopen for further investigation since we shouldn't end up with a bad cache.

Thank you for trying this out!

@alexr00 alexr00 closed this as completed Feb 23, 2022
@alexr00 alexr00 removed this from the March 2022 milestone Feb 23, 2022
@alexr00 alexr00 removed the bug Issue identified by VS Code Team member as probable bug label Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants