[CLEANUP] xmldoc package adds additional newlines in .csproj files about nx-dotnet HOT 9 OPEN

Hey @photomoose, I had not actually seen that the xmldoc toString method is not a guarantee. That's a shame, I would think it would be a main goal of an xml library.

The code looks much simpler with fast-xml-parser anyawys, and you've already got it sketched up. If you want to put in a PR for it go ahead, there are a few tests that should be updated as well.

from nx-dotnet.

If you do decide to work on it, go ahead and assign this issue to yourself.

from nx-dotnet.

Sure, I'll see what I can do. I don't appear to have permissions to assign this issue to myself, however - could you assign it to me, please?

from nx-dotnet.

Ah, my bad, I thought you could 😄. You are assigned.

Thanks for the contributions!

from nx-dotnet.

@photomoose Are you still planning to work on this? If not, I may pick it up.

from nx-dotnet.

from nx-dotnet.

@photomoose I'm going to tackle this this weekend, your initial investigations should provide a good starting point. Thanks!

from nx-dotnet.

@AgentEnder great stuff, I look forward to the update!

FYI: we now have 104 C# projects in our monorepo! We use Rider as our IDE and have imported all the projects into a single solution file, however we use NX Console in VS Code to create the projects via nx-dotnet.

from nx-dotnet.

Have you tried to add this options xmldoc.toString({ compressed: true, preserveWhitespace: true }). More information here nfarina/xmldoc#60.

from nx-dotnet.