VSDiagnostics
VSDiagnostics copied to clipboard
Vannevelj
A collection of static analyzers based on Roslyn that integrate with VS
The following members all start an infinite recursion that blows up in a `StackOverflowException`. ``` AnyType Identifier { get { return Identifier; } } AnyType Identifier => Identifier; AnyType Identifier...
I'm thinking about `*.Designer.cs` and `*.g.cs` for example.
Our tests are very bloated -- often we only test one or two specific lines of code whereas the rest of the test scenario stays in the same. Look into...
https://github.com/davidfowl/AspNetCoreDiagnosticScenarios/blob/master/AsyncGuidance.md
Stacktrace: ``` System.NullReferenceException : Object reference not set to an instance of an object. at async VSDiagnostics.Utilities.RenameHelper.RenameSymbolAsync() at async VSDiagnostics.Diagnostics.General.NamingConventions.NamingConventionsCodeFix.RenameAsync() at async Microsoft.CodeAnalysis.CodeActions.CodeAction.ComputeOperationsAsync() at async Microsoft.CodeAnalysis.CodeActions.CodeAction.ComputePreviewOperationsAsync() at async Microsoft.CodeAnalysis.CodeActions.CodeAction.GetPreviewOperationsAsync() at...
Is this really necessary? Maybe this one in particular should be an info instead.
Controller endpoints are a special case to exclude.
A lot of information can be found here: http://stackoverflow.com/questions/119506/virtual-member-call-in-a-constructor
I'm looking at `Extensions.IsAnInvocationOf`, `Extensions.IsAlias`, `Extensions.ToAlias`, `Extensions.IsNullable`, `Extensions.IsWhitespaceTrivia`, `Extensions.IsCommentTrivia`, `Extensions.ElementAtOrDefault` and `Extensions.IsNameofInvocation`