<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 18, 2019 at 3:03 AM Cleber Rosa <<a href="mailto:crosa@redhat.com">crosa@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 1/15/19 1:20 PM, Plamen Dimitrov wrote:<br>
> Hi all,<br>
> <br>
> I am new to the avocado-vt repo and it seems that merging pull requests<br>
> there has a different workflow than pull requests in avocado. In<br>
> particular, it seems that no review is required for a merge - am I<br>
> correct to assume this? I have 5-6 pull requests waiting and after<br>
> waiting for some time I just realized that GitHub is giving me green<br>
> light for each pull request without blocking. Shouldn't a main<br>
> maintainer also look at the code or is it good to go if it passes all<br>
> unit tests?<br>
> <br>
<br>
Avocado-VT workflow requires two ACKs, but you're right that the GH UI<br>
is not preventing merges.<br>
<br>
Xu, Lukáš (and others),<br>
<br>
Would you like to activate the merge block feature on GH?<br></blockquote><div><br></div><div>That sounds good to me, and honestly it is what I planned to do, so let's see other maintainers' willingness.</div><div><br></div><div>Thanks,</div><div>Xu<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
- Cleber.<br>
<br>
> Best,<br>
> Plamen<br>
> <br>
<br>
-- <br>
Cleber Rosa<br>
[ Sr Software Engineer - Virtualization Team - Red Hat ]<br>
[ Avocado Test Framework - <a href="http://avocado-framework.github.io" rel="noreferrer" target="_blank">avocado-framework.github.io</a> ]<br>
[  7ABB 96EB 8B46 B94D 5E0F  E9BB 657E 8D33 A5F2 09F3  ]<br>
</blockquote></div></div></div>