Paket should keep paket.dependencies as stable as possible

[forki]

Idea:

Instead of serializing the Dependencies file from the already parsed information we always modify the text and reparse it. Maybe not the best style, but it keep comments and everything in place.

fixes #780, #731

This needs a lot of testing. Especially the convert-from-nuget stuff (cc @theimowski)

/cc @matthid, @cjbhaines, @mexx, @inosik , @haf

[mexx]

Text representation have to be updated in this case.

[forki]

oups. correct.

Writing a test since this is uncovered.

[forki]

fixed in 93ca6f8

[theimowski]

I'll do some testing on Friday

[forki]

huh. I accidentally realigned the whole file. reverting that

[forki]

it's in 1.3.0-alpha001

[theimowski]

I think dependenciesFile is a good candidate for Property-Based round-trip test (parse, format, compare). Would need better tracking of comments/ custom code and similar stuff tho.

[forki]

would love to see that.

2015-04-30 21:07 GMT+02:00 Tomasz Heimowski notifications@github.com:

I think dependenciesFile is a good candidate for Property-Based
round-trip test (parse, format, compare). Would need better tracking of
comments/ custom code and similar stuff tho.

—
Reply to this email directly or view it on GitHub
#802 (comment).

[theimowski]

changes in NugetConver look ok - tested on a simple scenario, accidently also found a bug in the convert logic itself - fix in #804

[forki]

so ready to release?

[theimowski]