[Merge] lp:~sil2100/ubuntu-archive-tools/sru-stop-using-v-d into lp:ubuntu-archive-tools

Brian Murray brian at ubuntu.com
Thu Jun 8 19:39:49 UTC 2017


Review: Needs Fixing

One last change, sorry!

Diff comments:

> 
> === modified file 'sru-review'
> --- sru-review	2017-04-27 20:32:35 +0000
> +++ sru-review	2017-06-08 18:37:25 +0000
> @@ -346,14 +347,15 @@
>      text += ('Your feedback will aid us getting this update out to other '
>               'Ubuntu users.\n\nIf this package fixes the bug for you, '
>               'please add a comment to this bug, mentioning the version of the '
> -             'package you tested, and change the tag from verification-needed '
> -             'to verification-done. If it does not fix the bug for you, '
> -             'please add a comment stating that, and change the tag to '
> -             'verification-failed.  In either case, details of your testing '
> -             'will help us make a better decision.\n\nFurther information '
> -             'regarding the verification process can be found at '
> +             'package you tested, add the verification-done-%s tag for '
> +             'the series you have tested and remove verification-needed if '

I feel like the part "for the series you have tested" is rather redundant since the tag actually includes the series. So instead just "add the verification-done-zesty tag and remove verification-needed if..".

> +             'all available packages have been verified. If it does not fix '
> +             'the bug for you, please add a comment stating that, and change '
> +             'the tag to verification-failed.  In either case, details of your '
> +             'testing will help us make a better decision.\n\nFurther '
> +             'information regarding the verification process can be found at '
>               'https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  '
> -             'Thank you in advance!')
> +             'Thank you in advance!' % release)
>      bug.newMessage(content=text, subject='Please test proposed package')
>  
>  


-- 
https://code.launchpad.net/~sil2100/ubuntu-archive-tools/sru-stop-using-v-d/+merge/325326
Your team Ubuntu Package Archive Administrators is requested to review the proposed merge of lp:~sil2100/ubuntu-archive-tools/sru-stop-using-v-d into lp:ubuntu-archive-tools.



More information about the ubuntu-archive mailing list