Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit fde41e6

Browse files
tkirillshana
authored andcommitted
Fix the build
Add SuppressMessage attribute for unused field GuidList.guidContextMenuSet. This field breaks build of GitHub.VisualStudio project with error: > Running Code Analysis... >MSBUILD : error CA1823: Microsoft.Performance : It appears that field 'GuidList.guidContextMenuSet' is never used or is only ever assigned to. Use this field or remove it. > Code Analysis Complete -- 1 error(s), 0 warning(s) This field is already used in another branch (PR #156).
1 parent d9a1a90 commit fde41e6

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/GitHub.VisualStudio/Settings.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@ static class GuidList
1414

1515
public static readonly Guid guidGitHubCmdSet = new Guid(guidGitHubCmdSetString);
1616
public static readonly Guid guidGitHubToolbarCmdSet = new Guid(guidGitHubToolbarCmdSetString);
17+
18+
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Performance", "CA1823:AvoidUnusedPrivateFields", Justification = "Already used in https://github.com/github/VisualStudio/pull/156")]
1719
public static readonly Guid guidContextMenuSet = new Guid(guidContextMenuSetString);
1820
}
1921

0 commit comments

Comments
 (0)