{"action":"created","comment":{"url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/comments/47405185","id":47405185,"diff_hunk":"@@ -1,39 +1,142 @@\n-// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.\n+﻿// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.\n \n-using System.Collections.Immutable;\n-using Analyzer.Utilities;\n+using System.Linq;\n using Microsoft.CodeAnalysis.Diagnostics;\n+using System.Collections.Immutable;\n using Microsoft.CodeAnalysis;\n+using Analyzer.Utilities;\n+using Microsoft.CodeAnalysis.Semantics;\n \n namespace Microsoft.ApiDesignGuidelines.Analyzers\n-{                   \n+{\n     /// <summary>\n     /// CA1033: Interface methods should be callable by child types\n+    /// <para>\n+    /// Consider a base type that explicitly implements a public interface method.\n+    /// A type that derives from the base type can access the inherited interface method only through a reference to the current instance ('this' in C#) that is cast to the interface.\n+    /// If the derived type re-implements (explicitly) the inherited interface method, the base implementation can no longer be accessed.\n+    /// The call through the current instance reference will invoke the derived implementation; this causes recursion and an eventual stack overflow.\n+    /// </para>\n     /// </summary>\n-    public abstract class InterfaceMethodsShouldBeCallableByChildTypesAnalyzer : DiagnosticAnalyzer\n+    /// <remarks>\n+    /// This rule does not report a violation for an explicit implementation of IDisposable.Dispose when an externally visible Close() or System.IDisposable.Dispose(Boolean) method is provided.\n+    /// </remarks>\n+    [DiagnosticAnalyzer(LanguageNames.CSharp, LanguageNames.VisualBasic)]\n+    public sealed class InterfaceMethodsShouldBeCallableByChildTypesAnalyzer : DiagnosticAnalyzer\n     {\n         internal const string RuleId = \"CA1033\";\n \n         private static readonly LocalizableString s_localizableTitle = new LocalizableResourceString(nameof(MicrosoftApiDesignGuidelinesAnalyzersResources.InterfaceMethodsShouldBeCallableByChildTypesTitle), MicrosoftApiDesignGuidelinesAnalyzersResources.ResourceManager, typeof(MicrosoftApiDesignGuidelinesAnalyzersResources));\n-        \n         private static readonly LocalizableString s_localizableMessage = new LocalizableResourceString(nameof(MicrosoftApiDesignGuidelinesAnalyzersResources.InterfaceMethodsShouldBeCallableByChildTypesMessage), MicrosoftApiDesignGuidelinesAnalyzersResources.ResourceManager, typeof(MicrosoftApiDesignGuidelinesAnalyzersResources));\n         private static readonly LocalizableString s_localizableDescription = new LocalizableResourceString(nameof(MicrosoftApiDesignGuidelinesAnalyzersResources.InterfaceMethodsShouldBeCallableByChildTypesDescription), MicrosoftApiDesignGuidelinesAnalyzersResources.ResourceManager, typeof(MicrosoftApiDesignGuidelinesAnalyzersResources));\n-        \n-        internal static DiagnosticDescriptor Rule = new DiagnosticDescriptor(RuleId,\n-                                                                             s_localizableTitle,\n-                                                                             s_localizableMessage,\n-                                                                             DiagnosticCategory.Design,\n-                                                                             DiagnosticSeverity.Warning,\n-                                                                             isEnabledByDefault: false,\n-                                                                             description: s_localizableDescription,\n-                                                                             helpLinkUri: null,     // TODO: add MSDN url\n-                                                                             customTags: WellKnownDiagnosticTags.Telemetry);\n-\n-        public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);\n-\n-        public override void Initialize(AnalysisContext analysisContext)\n-        { \n-            \n+\n+        public static readonly DiagnosticDescriptor Rule = new DiagnosticDescriptor(RuleId,\n+                                                                          s_localizableTitle,\n+                                                                          s_localizableMessage,\n+                                                                          DiagnosticCategory.Design,\n+                                                                          DiagnosticSeverity.Warning,\n+                                                                          isEnabledByDefault: false,\n+                                                                          description: s_localizableDescription,\n+                                                                          helpLinkUri: \"https://msdn.microsoft.com/library/ms182153.aspx\",\n+                                                                          customTags: WellKnownDiagnosticTags.Telemetry);\n+\n+        public sealed override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);\n+\n+        public sealed override void Initialize(AnalysisContext analysisContext) =>\n+            analysisContext.RegisterOperationBlockAction(AnalyzeOperationBlock);\n+\n+        private bool ShouldExcludeOperationBlock(ImmutableArray<IOperation> operationBlocks)\n+        {\n+            if (operationBlocks != null && operationBlocks.Length == 1)\n+            {\n+                IBlockStatement block = operationBlocks[0] as IBlockStatement;\n+\n+                // An operation block that's not even a block - don't analyze the error cases.\n+                if (block == null)\n+                {\n+                    return true;\n+                }\n+\n+                if (block.Statements.Length == 0 ||\n+                    (block.Statements.Length == 1 && block.Statements[0].Kind == OperationKind.ThrowStatement))\n+                {\n+                    // Empty body OR body that just throws.\n+                    return true;\n+                }\n+            }\n+\n+            return false;\n+        }\n+\n+        private void AnalyzeOperationBlock(OperationBlockAnalysisContext context)\n+        {\n+            if (context.OwningSymbol.Kind != SymbolKind.Method)\n+            {\n+                return;\n+            }\n+\n+            var method = (IMethodSymbol)context.OwningSymbol;\n+\n+            // We are only interested in private explicit interface implementations within a public non-sealed type.\n+            if (method.ExplicitInterfaceImplementations.Length == 0 ||\n+                method.GetResultantVisibility() != SymbolVisibility.Private ||\n+                method.ContainingType.IsSealed ||\n+                method.ContainingType.GetResultantVisibility() != SymbolVisibility.Public)\n+            {\n+                return;\n+            }\n+\n+            // Avoid false reports from simple explicit implementations where the deriving type is not expected to access the base implementation.\n+            if (ShouldExcludeOperationBlock(context.OperationBlocks))\n+            {\n+                return;\n+            }\n+\n+            var hasPublicInterfaceImplementation = false;\n+            foreach (var interfaceMethod in method.ExplicitInterfaceImplementations)\n+            {\n+                // If any one of the explicitly implemented interface methods has a visible alternate, then effectively, they all do.\n+                if (HasVisibleAlternate(method.ContainingType, interfaceMethod))\n+                {\n+                    return;","path":"src/Microsoft.ApiDesignGuidelines.Analyzers/Core/InterfaceMethodsShouldBeCallableByChildTypes.cs","position":123,"original_position":123,"commit_id":"65e1e6245b20405540fab615ff23f2099aa35a0f","original_commit_id":"65e1e6245b20405540fab615ff23f2099aa35a0f","user":{"login":"srivatsn","id":20570,"avatar_url":"https://avatars.githubusercontent.com/u/20570?v=3","gravatar_id":"","url":"https://api.github.com/users/srivatsn","html_url":"https://github.com/srivatsn","followers_url":"https://api.github.com/users/srivatsn/followers","following_url":"https://api.github.com/users/srivatsn/following{/other_user}","gists_url":"https://api.github.com/users/srivatsn/gists{/gist_id}","starred_url":"https://api.github.com/users/srivatsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srivatsn/subscriptions","organizations_url":"https://api.github.com/users/srivatsn/orgs","repos_url":"https://api.github.com/users/srivatsn/repos","events_url":"https://api.github.com/users/srivatsn/events{/privacy}","received_events_url":"https://api.github.com/users/srivatsn/received_events","type":"User","site_admin":false},"body":"The loop is enumerating the explicit implementations of the *method* and not the *type*. In C# methods cannot have multiple explicit implementation but in VB one can write this:\r\n```vb.net\r\nSub Foo() Implements IFoo.Foo, IBar.Foo\r\n```\r\n\r\nSince this OperationAction is invoked once per method, your scenario will get covered anyway.","created_at":"2015-12-11T21:05:50Z","updated_at":"2015-12-11T21:05:50Z","html_url":"https://github.com/dotnet/roslyn-analyzers/pull/762#discussion_r47405185","pull_request_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762","_links":{"self":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/comments/47405185"},"html":{"href":"https://github.com/dotnet/roslyn-analyzers/pull/762#discussion_r47405185"},"pull_request":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762"}}},"pull_request":{"url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762","id":53342486,"html_url":"https://github.com/dotnet/roslyn-analyzers/pull/762","diff_url":"https://github.com/dotnet/roslyn-analyzers/pull/762.diff","patch_url":"https://github.com/dotnet/roslyn-analyzers/pull/762.patch","issue_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/762","number":762,"state":"open","locked":false,"title":"Port CA1033 as an IOperation analyzer","user":{"login":"srivatsn","id":20570,"avatar_url":"https://avatars.githubusercontent.com/u/20570?v=3","gravatar_id":"","url":"https://api.github.com/users/srivatsn","html_url":"https://github.com/srivatsn","followers_url":"https://api.github.com/users/srivatsn/followers","following_url":"https://api.github.com/users/srivatsn/following{/other_user}","gists_url":"https://api.github.com/users/srivatsn/gists{/gist_id}","starred_url":"https://api.github.com/users/srivatsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srivatsn/subscriptions","organizations_url":"https://api.github.com/users/srivatsn/orgs","repos_url":"https://api.github.com/users/srivatsn/repos","events_url":"https://api.github.com/users/srivatsn/events{/privacy}","received_events_url":"https://api.github.com/users/srivatsn/received_events","type":"User","site_admin":false},"body":"Move CA1033 from unfactored directory to ApiGuidelinesAnalyzer. The rule used to use a syntax action to find out if a codeblock was empty or if that just a single throw statement. That part has been rewritten to instead use and operation block that checks for the same conditions.\r\n\r\nVB has a bug whereby the operation tree includes an extra label statement that's causing some tests to fail and they have been skipped with the bug - https://github.com/dotnet/roslyn/issues/7397","created_at":"2015-12-11T01:24:12Z","updated_at":"2015-12-11T21:05:50Z","closed_at":null,"merged_at":null,"merge_commit_sha":"b2792b2efdb302b5985c21e1f6f8397f2ff3c544","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762/commits","review_comments_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762/comments","review_comment_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/comments{/number}","comments_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/762/comments","statuses_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/statuses/65e1e6245b20405540fab615ff23f2099aa35a0f","head":{"label":"srivatsn:ca1033","ref":"ca1033","sha":"65e1e6245b20405540fab615ff23f2099aa35a0f","user":{"login":"srivatsn","id":20570,"avatar_url":"https://avatars.githubusercontent.com/u/20570?v=3","gravatar_id":"","url":"https://api.github.com/users/srivatsn","html_url":"https://github.com/srivatsn","followers_url":"https://api.github.com/users/srivatsn/followers","following_url":"https://api.github.com/users/srivatsn/following{/other_user}","gists_url":"https://api.github.com/users/srivatsn/gists{/gist_id}","starred_url":"https://api.github.com/users/srivatsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srivatsn/subscriptions","organizations_url":"https://api.github.com/users/srivatsn/orgs","repos_url":"https://api.github.com/users/srivatsn/repos","events_url":"https://api.github.com/users/srivatsn/events{/privacy}","received_events_url":"https://api.github.com/users/srivatsn/received_events","type":"User","site_admin":false},"repo":{"id":45003660,"name":"roslyn-analyzers","full_name":"srivatsn/roslyn-analyzers","owner":{"login":"srivatsn","id":20570,"avatar_url":"https://avatars.githubusercontent.com/u/20570?v=3","gravatar_id":"","url":"https://api.github.com/users/srivatsn","html_url":"https://github.com/srivatsn","followers_url":"https://api.github.com/users/srivatsn/followers","following_url":"https://api.github.com/users/srivatsn/following{/other_user}","gists_url":"https://api.github.com/users/srivatsn/gists{/gist_id}","starred_url":"https://api.github.com/users/srivatsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srivatsn/subscriptions","organizations_url":"https://api.github.com/users/srivatsn/orgs","repos_url":"https://api.github.com/users/srivatsn/repos","events_url":"https://api.github.com/users/srivatsn/events{/privacy}","received_events_url":"https://api.github.com/users/srivatsn/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/srivatsn/roslyn-analyzers","description":"","fork":true,"url":"https://api.github.com/repos/srivatsn/roslyn-analyzers","forks_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/forks","keys_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/keys{/key_id}","collaborators_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/teams","hooks_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/hooks","issue_events_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/issues/events{/number}","events_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/events","assignees_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/assignees{/user}","branches_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/branches{/branch}","tags_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/tags","blobs_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/git/refs{/sha}","trees_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/git/trees{/sha}","statuses_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/statuses/{sha}","languages_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/languages","stargazers_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/stargazers","contributors_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/contributors","subscribers_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/subscribers","subscription_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/subscription","commits_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/commits{/sha}","git_commits_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/git/commits{/sha}","comments_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/comments{/number}","issue_comment_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/issues/comments{/number}","contents_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/contents/{+path}","compare_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/compare/{base}...{head}","merges_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/merges","archive_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/downloads","issues_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/issues{/number}","pulls_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/pulls{/number}","milestones_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/milestones{/number}","notifications_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/labels{/name}","releases_url":"https://api.github.com/repos/srivatsn/roslyn-analyzers/releases{/id}","created_at":"2015-10-26T23:17:27Z","updated_at":"2015-10-28T19:29:16Z","pushed_at":"2015-12-11T19:01:59Z","git_url":"git://github.com/srivatsn/roslyn-analyzers.git","ssh_url":"git@github.com:srivatsn/roslyn-analyzers.git","clone_url":"https://github.com/srivatsn/roslyn-analyzers.git","svn_url":"https://github.com/srivatsn/roslyn-analyzers","homepage":null,"size":4923,"stargazers_count":0,"watchers_count":0,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"dotnet:master","ref":"master","sha":"ec6aff188a7cd49a1bf8285c7741b8413cdbe260","user":{"login":"dotnet","id":9141961,"avatar_url":"https://avatars.githubusercontent.com/u/9141961?v=3","gravatar_id":"","url":"https://api.github.com/users/dotnet","html_url":"https://github.com/dotnet","followers_url":"https://api.github.com/users/dotnet/followers","following_url":"https://api.github.com/users/dotnet/following{/other_user}","gists_url":"https://api.github.com/users/dotnet/gists{/gist_id}","starred_url":"https://api.github.com/users/dotnet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dotnet/subscriptions","organizations_url":"https://api.github.com/users/dotnet/orgs","repos_url":"https://api.github.com/users/dotnet/repos","events_url":"https://api.github.com/users/dotnet/events{/privacy}","received_events_url":"https://api.github.com/users/dotnet/received_events","type":"Organization","site_admin":false},"repo":{"id":36946704,"name":"roslyn-analyzers","full_name":"dotnet/roslyn-analyzers","owner":{"login":"dotnet","id":9141961,"avatar_url":"https://avatars.githubusercontent.com/u/9141961?v=3","gravatar_id":"","url":"https://api.github.com/users/dotnet","html_url":"https://github.com/dotnet","followers_url":"https://api.github.com/users/dotnet/followers","following_url":"https://api.github.com/users/dotnet/following{/other_user}","gists_url":"https://api.github.com/users/dotnet/gists{/gist_id}","starred_url":"https://api.github.com/users/dotnet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dotnet/subscriptions","organizations_url":"https://api.github.com/users/dotnet/orgs","repos_url":"https://api.github.com/users/dotnet/repos","events_url":"https://api.github.com/users/dotnet/events{/privacy}","received_events_url":"https://api.github.com/users/dotnet/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/dotnet/roslyn-analyzers","description":"","fork":false,"url":"https://api.github.com/repos/dotnet/roslyn-analyzers","forks_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/forks","keys_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/teams","hooks_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/hooks","issue_events_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/events{/number}","events_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/events","assignees_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/assignees{/user}","branches_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/branches{/branch}","tags_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/tags","blobs_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/git/refs{/sha}","trees_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/statuses/{sha}","languages_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/languages","stargazers_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/stargazers","contributors_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/contributors","subscribers_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/subscribers","subscription_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/subscription","commits_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/commits{/sha}","git_commits_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/git/commits{/sha}","comments_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/comments{/number}","issue_comment_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/comments{/number}","contents_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/contents/{+path}","compare_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/merges","archive_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/downloads","issues_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues{/number}","pulls_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls{/number}","milestones_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/milestones{/number}","notifications_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/labels{/name}","releases_url":"https://api.github.com/repos/dotnet/roslyn-analyzers/releases{/id}","created_at":"2015-06-05T17:48:07Z","updated_at":"2015-11-30T02:11:13Z","pushed_at":"2015-12-11T20:46:09Z","git_url":"git://github.com/dotnet/roslyn-analyzers.git","ssh_url":"git@github.com:dotnet/roslyn-analyzers.git","clone_url":"https://github.com/dotnet/roslyn-analyzers.git","svn_url":"https://github.com/dotnet/roslyn-analyzers","homepage":null,"size":6610,"stargazers_count":60,"watchers_count":60,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":53,"mirror_url":null,"open_issues_count":181,"forks":53,"open_issues":181,"watchers":60,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762"},"html":{"href":"https://github.com/dotnet/roslyn-analyzers/pull/762"},"issue":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/762"},"comments":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/issues/762/comments"},"review_comments":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762/comments"},"review_comment":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/pulls/762/commits"},"statuses":{"href":"https://api.github.com/repos/dotnet/roslyn-analyzers/statuses/65e1e6245b20405540fab615ff23f2099aa35a0f"}}}}