← Back to team overview

credativ team mailing list archive

Re: [Merge] lp:~therp-nl/openupgrade-addons/7.0-project_data into lp:openupgrade-addons

 

I would rather keep the rules, so that we get a perfect upgrade. Otherwise, the visibility field on the project will not work as expected.

As for the formatting, it is purely cosmetic. The output is generated from a diff on xml level and I would prefer not to change too much of it. I reformatted indentation, but the contents are perfectly valid, so I'm asking you to let this pass too. Whitespace in domains is a matter of taste. Personally, I actually like this better than the original representation! I will try to integrate the xml-diff script into the analysis phase at some point, and the productivity gains of that would mostly be lost if we now start to do extensive reformatting on its contents.



-- 
https://code.launchpad.net/~therp-nl/openupgrade-addons/7.0-project_data/+merge/216538
Your team OpenUpgrade Committers is subscribed to branch lp:openupgrade-addons.


References