On Mon, May 3, 2010 at 8:24 AM, Martin Pool <email address hidden> wrote:
> On 3 May 2010 12:39, Parth Malwankar <email address hidden> wrote:
>> The docstring in the script does mention.
>>
>> * this script will only work when run by members of ~canonical,
>> because it's a private team
>>
>> I thought that might have caused this wasn't sure. I suppose the
>> easiest fix would be to just give a clearer error message.
>>
> Perhaps if you're not allowed to see that team (ie you get a KeyError)
> you should continue without it, but print a message along with the
> list at the end.
>
On Mon, May 3, 2010 at 8:24 AM, Martin Pool <email address hidden> wrote:
> On 3 May 2010 12:39, Parth Malwankar <email address hidden> wrote:
>> The docstring in the script does mention.
>>
>> * this script will only work when run by members of ~canonical,
>> because it's a private team
>>
>> I thought that might have caused this wasn't sure. I suppose the
>> easiest fix would be to just give a clearer error message.
>>
> Perhaps if you're not allowed to see that team (ie you get a KeyError)
> you should continue without it, but print a message along with the
> list at the end.
>
Sounds like a good solution to me.
Regards,
Parth