Robin Sue

- Except for the PascalCase refactoring, i think we've got this covered. For the PascalCase refactoring, one could also ship it in the core package and turn down the default...

Consistent naming is something that bothered me for some while now. We are, for example, only rarely using destructuring on complex types which makes this idea unsuitable for many usecases...

@kevygreen You could start around in this region for the analysis part: https://github.com/Suchiman/SerilogAnalyzer/blob/b78fb7f1e0b7de65bd9b46ba3b17d62d445c4288/SerilogAnalyzer/SerilogAnalyzer/DiagnosticAnalyzer.cs#L170-L239 The configuration bit is tricky, roslyn has nothing to offer for this, but i've been thinking that...

@normanhh3 there's no builtin analyzer mechanism for maintaining state. Depends on what sort of analyzer you write, usually they operate on one file or one syntax location though and they...

I'm not very familiar with `Microsoft.Extensions.Logging` but i will have a look if the whitelist can be simply expanded to include it or if `Microsoft.Extensions.Logging` requires different checks than Serilog.

@prlcutting Yes, i'll gladly take a PR!

@hellboy81 SerilogAnalyzer is a plain roslyn analyzer, you can install it as a nuget package to get build time warnings but last time i've checked, omnisharp still doesn't support using...

I don't think any of that is necessary, `SerilogAnalyzer.dll` is neither copied to the output directory nor are you able to access the `SerilogAnalyzer` namespace in your project where you...

Can you post a snippet on which it does not work?

It does work for me on this snippet, you can place the cursor anywhere between `new` and `;`. Are you using VS (2017 / 2019?) or VS Code? Nuget version...