Josef Pihrt
Josef Pihrt
Hi, The configuration file at the extension directory is not meant to be edited by users. Users should use file located at `%LOCALAPPDATA%/JosefPihrt/Roslynator/.roslynatorconfig`.
I'm not sure that this proposal fits into this analyzer. The purpose of this analyzer is simply to remove (or let's say unwrap) last else (in if-else sequence). To be...
> The Rider analogue of this analyzer has exactly the behaviour @jamesHargreaves12 Could you send me a link to their docs? I would like to take a closer look at...
@jamesHargreaves12 ping
Analyzer from package Roslynator.CodeAnalysis.Analyzers were never included in the VS extension. These analyzers are relevant only for projects that reference Roslyn packages (Microsoft.CodeAnalysis*) https://josefpihrt.github.io/docs/roslynator/analyzers#groups
@orels1 The extension with analyzers will be still available as GH artifact as it can be seen [here](https://github.com/dotnet/roslynator/actions/runs/7690128207?pr=1303). Would that do?
@orels1 It will be documented, no doubt about it.
@jamesHargreaves12 MinBy/MaxBy throws exception in case of no elements so it should be First not FirstOrDefault.
Ok, it does not throw always when the sequence is empty but sometimes it does: https://source.dot.net/#System.Linq/System/Linq/Max.cs,442 ```cs /// If is a reference type and the source sequence is empty or...
You are right that the doc is wrong, but the bottom line is that MaxBy throws sometimes which must be taken into account when applying these LINQ optimizations.