Lasath Fernando
Lasath Fernando
You could just add both binaries. (Or maybe a symlink) That way you can add the new functionality without breaking anything.
Is that really necessary? I feel like csharpier is already bloody fast. :stuck_out_tongue:
Lambda in method formatting should probably be a separate issue than lambda formatting. So, it should probably be something closer to: ``` entity.HasKey( e => new { e.BusinessEntityID, e.PersonID, e.ContactTypeID...
You think something like this might look better? To make it clear that the braces aren't for the outer level statement, but rather the lambda being passed in? ```cs var...
Adding a link to [the file](https://github.com/belav/csharpier/blob/a4ba7787f00cb63e90bd821770860df82262ab37/Src/CSharpier.Tests/TestFiles/ObnoxiousEdgeCases/ObnoxiousEdgeCases.cst), for easy reference.
I think ```cs Group( Group("public virtual", "ICollection"), Group("SomeLongNameThatForcesALineBreak", "{ get; set; } ="), "HashSet" ) ``` (with the contents in "" being printed recursively by syntax printers, of course) would...
Don't have a super strong preference, but I'm curious how you'd handle expression bodied lambdas like: ```cs someLongAssObject.SomeLongAssFunction((someLongAssArg, someLongerAsserArg) => someLongAssArg.SomeLongAssMethod(someLongerAsserArg.SomeLongAssProperty)) ```
I brought this up before with them, but didn't get any response: https://github.com/dotnet/roslyn/discussions/54099 When I pinged the roslyn devs offline, they were against the idea since csharpier does't handle code...
> 3. Before writing the file, reread it and merge the two results. > > 3 seems like the best option to me, and I believe it will be relatively...
Good point. But there is a windows equivalent: https://github.com/danieljoos/wincred