Code review comment for lp:~mandel/ubuntuone-client/eq-add-ancestors-watches
Revision history for this message
| Alejandro J. Cura (alecu) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Some comments on the style, while I finish the review:
There are a few dangling space characters at the end of some lines.
Also, a few typos:
"a new istance" - > "a new instance"
"ar not added" -> "are not added"
"have not bee" -> "have not been"
"for a use to" -> "for a user to"
"On windows we do no" -> "On windows we do not"
"we need to ensure that the udf is not watch because there was an issue when dealing with the addition of the parent watches." -> "we need to ensure that the udf is not subscribed when an error happens while adding the parent watches."