[almighty] Code review integration

[Shoubhik Bose]

"Reviewable" webhook seems to be active yet again in almighty/almighty-core
, any clue why it was done? :)

On Mon, Sep 19, 2016 at 8:12 PM, Todd Mancini <tmancini at redhat.com> wrote:

> Lack of sleep and context did not help here. Ignore me.
>
> On Sun, Sep 18, 2016 at 8:11 PM, Aslak Knutsen <aslak at redhat.com> wrote:
>
>> Don't listen to Todd, his answering in his sleep... :)
>>
>> The main reason we moved to Reviewable was to keep track of 'unanswered'
>> comments during large reviews. I'm not seeing 100% how the new GitHub
>> review fixes this, but, the new GitHub Review definitely has nicer reviews
>> than before and with generally smaller pull requests I think we should be
>> able to use it with no problem.
>>
>> My take would be; Let's try GitHub Review for a while and see if it works
>> for what we need. If not, we can move back to Reviewable later.
>>
>> -aslak-
>>
>> On Sun, Sep 18, 2016 at 9:06 PM, Todd Mancini <tmancini at redhat.com>
>> wrote:
>>
>>> Yes, we should continue with our implementation. There are many reasons,
>>> but the main reason is that, although we are using GitHub today, the future
>>> plans do not require GitHub.
>>>
>>> Sent from my phone, so anticipate hilarious autocorrects
>>> ------------------------------
>>> From: Shoubhik Bose <shbose at redhat.com>
>>> Sent: ‎9/‎16/‎2016 2:57 AM
>>> To: ALMighty-public <almighty-public at redhat.com>
>>> Subject: [almighty] Code review integration
>>>
>>> Github has added new features for tracking code review , should we be
>>> continuing with reviewable ?
>>>
>>> _______________________________________________
>>> almighty-public mailing list
>>> almighty-public at redhat.com
>>> https://www.redhat.com/mailman/listinfo/almighty-public
>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/almighty-public/attachments/20170102/23c7a618/attachment.htm>