W3C home > Mailing lists > Public > public-webapps-testsuite@w3.org > September 2013

Re: Need reviewers for Cameron's Web IDL tests

From: Simon Pieters <simonp@opera.com>
Date: Mon, 23 Sep 2013 15:54:02 +0200
To: "Arthur Barstow" <art.barstow@nokia.com>
Cc: "public-webapps-testsuite@w3.org" <public-webapps-testsuite@w3.org>, public-script-coord <public-script-coord@w3.org>
Message-ID: <op.w3u98cb7idj3kv@simons-macbook-pro.local>
On Mon, 23 Sep 2013 15:30:55 +0200, Arthur Barstow <art.barstow@nokia.com>  
wrote:

> On 9/20/13 4:35 PM, ext Simon Pieters wrote:
>> On Tue, 17 Sep 2013 13:10:19 +0200, Arthur Barstow  
>> <art.barstow@nokia.com> wrote:
>>
>>> Although we expect all review feedback to be submitted as comments to  
>>> PR-271, we are using [Tracking-271] to track our review progress.
>>
>> I'm a bit curious why you don't use  
>> https://critic.hoppipolla.co.uk/r/247 for handling assignments, review  
>> comments and tracking progress?
>>
>
> I prefer to use Github for the review comments.

OK.

> Using a wiki to track the assignments and progress seemed like an easy  
> solution although I suppose we could have GH for that too.
>
> BTW, what do I need to do to stop getting e-mails from  
> critic.hoppipolla.co.uk (to my afbarstow @ gmail.com account) that only  
> announce a new PR was created? For example, I don't want to get these  
> types of announcements:
>
> [[
> Critic review: https://critic.hoppipolla.co.uk/r/318
>
> This is an external review system which you may optionally use for the  
> code review of your pull request.
> ]]

I think you can uncheck email.activated in  
https://critic.hoppipolla.co.uk/config

-- 
Simon Pieters
Opera Software
Received on Monday, 23 September 2013 13:54:38 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 6 January 2015 19:57:41 UTC