The Wayback Machine - https://web.archive.org/web/20201215060029/https://github.com/github/VisualStudio/issues/2542
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

Pull request comments not visible inline for VS2017 #2542

Open
dtuckerFort opened this issue Sep 3, 2020 · 4 comments
Open

Pull request comments not visible inline for VS2017 #2542

dtuckerFort opened this issue Sep 3, 2020 · 4 comments

Comments

@dtuckerFort
Copy link

@dtuckerFort dtuckerFort commented Sep 3, 2020

Versions

  • GitHub Extension for Visual Studio version: ... 2.11.104.25275
  • Visual Studio version: ... VS Professional 2017 15.9.26

What happened

The GitHub plugin let's me add a pull-request comment inline but doesn't show any comments inline. Comments are only visible within the Reviewer view. My teammates have the same problem. Sometimes it works, sometimes it doesn't

extension.log when clicking link to open PR comment:
2020-09-03 17:05:41.240 [43780] EROR [01] PullRequestDetailViewModel Error loading pull request [local git repo]
System.NullReferenceException: Object reference not set to an instance of an object.
at GitHub.ViewModels.GitHubPane.PullRequestDetailViewModel.d__111.MoveNext() in C:\projects\visualstudio\src\GitHub.App\ViewModels\GitHubPane\PullRequestDetailViewModel.cs:line 0
2020-09-03 17:12:06.123 [07420] EROR [61] PullRequestSessionManager
System.AggregateException: One or more errors occurred. ---> System.NullReferenceException: Object reference not set to an instance of an object.
at GitHub.InlineReviews.Services.PullRequestSessionService.d__24.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionService.cs:line 368
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__18.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 249
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__17.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 214
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__16.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 193
--- End of inner exception stack trace ---
---> (Inner Exception #0) System.NullReferenceException: Object reference not set to an instance of an object.
at GitHub.InlineReviews.Services.PullRequestSessionService.d__24.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionService.cs:line 368
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__18.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 249
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__17.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 214
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHub.InlineReviews.Services.PullRequestSessionManager.d__16.MoveNext() in C:\projects\visualstudio\src\GitHub.InlineReviews\Services\PullRequestSessionManager.cs:line 193<---

extension.log when clicking link to a closed PR comment:
"PullRequestReviewCommentViewModel Couldn't find line number for comment on"

This seems to occur when a user makes posts a comment directly on the GitHub website.

@jxz-mdclarity
Copy link

@jxz-mdclarity jxz-mdclarity commented Sep 29, 2020

I have the same issue with both Visual Studio 2017 Pro and 2019 Pro.

GitHub Extension v2.11.106.19330
VS Pro 2017 15.9.27
VS Pro 2019 16.7.4

Not sure how to set the priority of this issue (as I am relatively new to GitHub), but this is hitting the productivity of our code review big time.

@dtuckerFort
Copy link
Author

@dtuckerFort dtuckerFort commented Oct 5, 2020

This seems to occur when a user makes posts a comment directly on the GitHub website.

@jxz-mdclarity
Copy link

@jxz-mdclarity jxz-mdclarity commented Oct 5, 2020

@dtuckerFort One collaborator and I myself post comments through the Visual Studio's GitHub extension, but I think another collaborator used the website to post comment too.

@stalek71
Copy link

@stalek71 stalek71 commented Nov 26, 2020

It doesn't work with inline comments also... I created some comment when I was reviewing PR changes and it appeared on GitHub website without any problems but clicking it on Reviews subtab in GitHub extension(tab) resulted in providing the same error as usual.

At the moment it's just useless ... :(

PullRequestReviewCommentViewModel Couldn't find line number for comment on Some/Path/To/File.h @ 58e6e9ac17789dd89eb81debcc46d5c84fa47b86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.