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.
« Back to merge proposal
Hi Leo,
Awesome, looking good.
Only one minor thing to ask of you: pick_app_ launcher_ produces_ deprecation_ message here: /code.launchpad .net/~veebers/ autopilot/ fix_deprecate_ pick_app_ launcher/ +merge/ 202784)
Please add a test for L275 to ensure the deprecated message is output.
(An example can be found within
test_using_
https:/
Once this is done I'll be more than happy to approve.
Thanks again.