Adds NewLinesForBracesInProperties option.#232
Adds NewLinesForBracesInProperties option.#232mkrueger wants to merge 1 commit intodotnet:masterfrom mkrueger:issue84
Conversation
Fixes Issue #84
|
@dotnet-bot retest this, please. |
|
The problem is that I can't run roslyn from inside md :/. Treat that as proposal. |
|
I would think it also needs to include a corresponding change to NewLinesViewModel.cs and a few other places? |
|
@mkrueger - what do you think - can you add some tests for this? |
|
If you can make monodevelop loading the project + the ms tests :/. ATM I have no way running it. |
|
We don't have any mstest based tests, we use xunit. You should be able to install the xunit.runners package (done as part of (Also spoke to @jaredpar - we just just restore the runners package directly instead of waiting for you to run MSBuild BuildAndTest.pro) |
|
I'll look at it. UPDATE: I still can't load the roslyn project with monodevelop - will try again after project system changes. |
|
Note that there is no contributor agreement for this PR. It is also quite stale. |
|
We still have problems with the roslyn project on monodevelop :/ - as said regard that as proposal. |
Bumps [Microsoft.SourceLink.GitHub](https://github.com/dotnet/sourcelink) from 1.1.1 to 8.0.0. - [Release notes](https://github.com/dotnet/sourcelink/releases) - [Commits](dotnet/sourcelink@1.1.1...8.0.0) --- updated-dependencies: - dependency-name: Microsoft.SourceLink.GitHub dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Fixes Issue #84.
Just a possible way of handling the formatting. (Howver unit tests are missing and should be added)