Code review comment for lp:~elopio/autopilot/no_uinput_side-effects

Revision history for this message
Christopher Lee (veebers) wrote :

Hi Leo,

Awesome, looking good.

Only one minor thing to ask of you:
Please add a test for L275 to ensure the deprecated message is output.
(An example can be found within
test_using_pick_app_launcher_produces_deprecation_message here:
https://code.launchpad.net/~veebers/autopilot/fix_deprecate_pick_app_launcher/+merge/202784)

Once this is done I'll be more than happy to approve.

Thanks again.

review: Needs Fixing

« Back to merge proposal