Lasath Fernando

Results 27 comments of 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