dazinator / AspNetSolutionUpgradeTool

A utility that can be used to upgrade an ASP.NET 5 RC1 or RC2 based solution to ASP.NET Core 1.0.0 RTM.
MIT License
5 stars 2 forks source link

Please prettyprint project.json #30

Closed ghost closed 8 years ago

ghost commented 8 years ago

After converting a hello world project on Windows the project.json file is ONE long line which is difficult to inspect and maintain. It would be nice if it was formatted with line breaks and indent like the source project.json file was.

dazinator commented 8 years ago

yeah I noticed this but haven't addressed it yet! In order to work around this, I have been going back into them after in VS, and using ctrl + k + d for format the document. Not ideal at all!

ghost commented 8 years ago

What tool are you using to output json? For example, newtonsoft has an option to prettyprint things that you could easily use.

dazinator commented 8 years ago

That should be fixed now, I have adjusted the approval tests to catch regressions with formatting in future. The actual change I made was a very simple one liner:

https://github.com/dazinator/AspNetRC1toRC2UpgradeTool/commit/0c92e540f3ef40d96a7a1bef879a7d9011772149#diff-57fd8ecb3265e705bfdbee791a4fcbafR46

ghost commented 8 years ago

Nice!