Code review comment for lp:~ubuntu-weather-dev/ubuntu-weather-app/reboot-finish-listitem-migration

Revision history for this message
Victor Thompson (vthompson) wrote :

> Should this have // CUSTOM as a note incase we do a pull of upstream and miss that we made changes?
Sure. I added a comment. I think eventually this will be folded into the SDK anyway. They already pulled in a hint component that we should make use of soonish.

> Why was this changed it looks the same just with { } around it?
No reason that I know of. I didn't like it either when I was resolving merge conflicts. Updated.

> Not A->Z
Fixed.

« Back to merge proposal