search

belav / csharpier

CSharpier is an opinionated code formatter for c#.
https://csharpier.com
MIT License
1.41k stars 98 forks source link

VS Extension now hot reloads its config #1369

Closed scabana closed 2 weeks ago

scabana commented 2 weeks ago

This fixes #1367.

The .vs/$(SolutionName)/v17/csharpier.json file is now watched for changes and will reload the configuration when the file is changed.

belav commented 2 weeks ago

LGTM. You've tested this thoroughly, right?

Neither @belav or I use Visual Studio, so we won't really notice regressions very quickly.

@shocklateboy92 I tested all the test cases I could think of, the code seems pretty straight forward.

scabana commented 2 weeks ago

Hey, yeah, checked new code does as intended. Also validated updating the config from the UI worked and didn't trigger a chain reaction.

Sorry for the format issue, I thought I had formatted it.

belav commented 2 weeks ago

Sorry for the format issue, I thought I had formatted it.

No worries! I feel like I somehow miss formatting a file in almost every PR. Thanks for your contribution!