| nigelb | bdmurray, I did some edits to the patch subscription script and pushed to my own branch. can take a look? | 01:44 |
|---|---|---|
| nigelb | http://bazaar.launchpad.net/~nigelbabu/ubuntu-qa-tools/patch-review-script/annotate/head%3A/launchpadlib-scripts/patch-subscriber.py | 01:44 |
| bdmurray | nigelb: seems fine but I don't see any reason to reduce the list of workflow_teams as they might be subscribed to old bug reports and it doesn't hurt does it? | 01:51 |
| nigelb | bdmurray, I wanted to talk to the old workflow teams and help extend dholbach's resubscribe script for those teams too | 01:51 |
| nigelb | like subscribe ubuntu-sru where motu-sru is subscribed etc, persia_ encouraged me to talk to the team owners about that | 01:52 |
| nigelb | and i've added one more package to jfo's list and one more package to be ignored :) | 01:53 |
| bdmurray | okay but if an old team is subscribed to a bug with a patch and then the reviewers team gets subscribed to it what will the reviewers team do? | 01:53 |
| bdmurray | nigelb: yes I saw those other changes too thanks! | 01:53 |
| nigelb | No, what I meant is, daniel's script subscribes sponsors if u-u-s or u-m-s is subscribed to a bug | 01:54 |
| nigelb | so if we can give the old teams an extend of that script to be run occasionally, they can take care of it | 01:55 |
| nigelb | we might not have anything to review on a motu-sru bug, we'll just delay it | 01:55 |
| bdmurray | right and if a bug with u-u-s subscribed shows up in our search results with your branch the reviewers team will be subscribed to it | 01:55 |
| bdmurray | because your branch removes u-u-s from workflow teams | 01:56 |
| nigelb | aah. ok. common sense. I should have left them there :D | 01:56 |
| bdmurray | so if you fix that I'll be happy to merge it | 01:56 |
| nigelb | ubuntu-ma-sponsors should be ubuntu-main-sponsors? | 01:58 |
| bdmurray | why yes! | 01:59 |
| nigelb | ok, so here goes | 02:02 |
| nigelb | http://bazaar.launchpad.net/~nigelbabu/ubuntu-qa-tools/patch-review-script/annotate/head%3A/launchpadlib-scripts/patch-subscriber.py | 02:02 |
| nigelb | shall I request a merge? | 02:02 |
| dholbach | good morning | 07:16 |
| === dholbach_ is now known as dholbach | ||
| nigelb | I'm slowly working on the https://wiki.ubuntu.com/ReviewersTeam/GettingInvolved to incorporate examples. If someone would like to give suggestions, correction, please let me know | 15:15 |
| nigelb | hyperair, poke | 15:16 |
| === yofel_ is now known as yofel | ||
| nigelb | bdmurray, you want me to request a sync for the script? | 17:11 |
| bdmurray | nigelb: a merge proposal would help thanks | 17:12 |
| nigelb | bdmurray, will do. I wasn't sure how to go about it :) | 17:12 |
| nigelb | persia, https://wiki.ubuntu.com/ReviewersTeam/GettingInvolved | 17:35 |
| nigelb | any more suggestions? | 17:35 |
| nigelb | bdrung, got a minute? | 17:40 |
| bdrung | nigelb: at least three seconds ;) | 17:41 |
| bdrung | *at most | 17:41 |
| nigelb | can sign up for a few hours at https://wiki.ubuntu.com/PatchDay? | 17:41 |
| nigelb | bdrung, ^ | 17:43 |
| bdrung | nigelb: will have a look at it later | 17:48 |
| nigelb | bdrung, thank you :) even 1 hour is much appreciated :) | 17:48 |
| * nigelb notes that there is still 40 slots open | 18:24 | |
| Ciemon | nigelb: I just had a look at PatchDay, I wonder if that's something for someone with minimal experience.. like me? | 18:42 |
| nigelb | Ciemon, yep it is. you're welcome to contribute | 18:43 |
| nigelb | you need to be able to test patches and if working, encourage patch author to forward to upstream bug tracker | 18:43 |
| nigelb | Ciemon, the reviewers team doesn't have too much requirements, see https://wiki.ubuntu.com/ReviewersTeam | 18:44 |
| Ciemon | OK, I'll have had more practice my then; and I see the "bugs with patches added" link. | 18:44 |
| Ciemon | yeah, I've read ReviewersTeam before.. interesting that you've said that the patch author shold forward upstream. Is that "policy"? | 18:46 |
| Ciemon | It's something I want to make happen with the batch I'm working through at the moment. | 18:46 |
| nigelb | Ciemon, the idea is the author should be able to respond to feedback | 18:46 |
| nigelb | Ciemon, but if patch author shows no response, we'll have to forward | 18:47 |
| * Ciemon ponders | 18:47 | |
| nigelb | Ciemon, upstream might ask for changes or give comments, its easier if the patch author does it him/her-self | 18:49 |
| Ciemon | ok | 18:50 |
| nigelb | I'm off. ask any questions here, I'll reply in scroll back if noone else does :) | 18:51 |
| Ciemon | Thanks.. no questions for now, just practise. | 18:54 |
| === BlackZ_ is now known as BlackZ | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!