Jeroen Vannevel

JSX support

4

First of all, thank you for this great library! It's been a delight to work with, both for its speed as well as the easy-to-use API it exposes. I'm aware...

Panic when using Fixer: assertion failed: `(left == right)`

3

I'm using rslint's `Fixer` to make changes in source code but have found that certain order of operations cause an internal panic. I have narrowed it down to this: ```rs...

Default function parameters are incompletely parsed as AssignPattern

1

It appears as if default function values are parsed in an unexpected manner. Specifically, the `AssignPattern.key()` value always returns `None`, when I would expect it to be a `SinglePattern`. I...

I'm the author of both, VSDiagnostics hasn't been updated in years. SharpSource is actively being worked on though.

🐛 Bug: Tests default to full stacktrace on Windows, rather than shortened

8

I'm using mocha 2.3.3 which should have the shortened stacktraces by default yet I am still receiving them in full: ``` 2) conditionBuilder greater_than_condition returns false with smaller input: AssertionError:...

Don't display warnings for generated files

5

I'm thinking about `*.Designer.cs` and `*.g.cs` for example.

Prettify tests

1

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.