Code review comment for lp:~robert-ancell/indicator-session/unity-settings-daemon
Revision history for this message
| Dimitri John Ledkov (xnox) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
f33e652
(Get the code!)
DONE * Ensure the project compiles and the test suite executes without error
DONE * Ensure that non-obvious code has comments explaining it
DONE * If the change works on specific profiles, please include those in the merge description. (no profile specific changes)
DONE * Did the Jenkins build compile? Pass? Run unit tests successfully?
DONE * Are there appropriate tests to cover any new functionality?