Florian Verdonck

Results 874 comments of Florian Verdonck

Another nitpick here, but ![image](https://github.com/fsharp/FsAutoComplete/assets/2621499/133d929d-eea1-456a-8d87-4b96d03c3615) can be avoided by not using `cramped` for https://fsprojects.github.io/fantomas/docs/end-users/Configuration.html#fsharp_multiline_bracket_style. Would you be on board to switch to `aligned` or `stroustrup` for either these two files...

@TheAngryByrd which one do you prefer?

Yeah, this is something we want to do after the nightly (`8.0.300`) is merged back into main. Too many changes there. I also found https://github.com/fsprojects/fantomas/issues/3070

I still need to improve this PR a bit regardless of formatting. I would do format configuration changes in a separate PR.

@brianrourkeboll would you mind taking a peek at https://github.com/ionide/FsAutoComplete/pull/1252/commits/90f17fa82e59a6ff42efd1b702bcfbf7fb66a913 I had to change some tests, so I'm probably doing something wrong.

> The parens in those tests are indeed not needed. https://github.com/dotnet/fsharp/pull/16973 isn't available yet, is it? No, it is not available in the nightlies. But I wasn't sure if that...

Hello, I'm not quite sure why this isn't showing up. I think you would need to debug the assembly processing to see what is up.

More things happen than they should when touching a file, indicating deeper structural issues. I'm open to suggestions, but proceed with caution.

Hi, > I needed to add a new pipeline I want to push back on that a little. Once we merge, it's on us to maintain, so we need to...

Hello, I think this is a reasonable request. I'm not aware of any strong attachments to our current setup. As long as the out-of-the-box experience works, we can revisit this....