<div dir="ltr"><div dir="ltr"><div dir="ltr">Thanks very much Marco and welcome to the team! ;-)</div><div dir="ltr"><br></div><div dir="ltr">I've added your name to the rota. I'm in the UK so let's chat on irc/slack when you're about next week.<div><br></div><div>Here's the full PR backlog (but don't let it put you off :)...</div><div><br></div><div>- <a href="https://github.com/pulls?utf8=%E2%9C%93&q=is%3Aopen+is%3Apr+org%3Akata-containers">https://github.com/pulls?utf8=%E2%9C%93&q=is%3Aopen+is%3Apr+org%3Akata-containers</a></div><div><br></div><div>Personally, I tend to "round robin" through the repos and crawl over the list of open PRs in each.</div><div><br></div><div>If anyone else wants to join in, feel free to join us!</div><div><br></div><div>Bottom line: the more PRs we can merge, the more bug fixes, optimisations and juicy new features we all get to benefit from :)</div><div><br></div><div>Cheers,<br></div><div><br></div><div>James</div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-08-31 15:56 GMT+01:00 Marco Vedovati <span dir="ltr"><<a href="mailto:mvedovati@suse.com" target="_blank">mvedovati@suse.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi James,<br>
thanks for this great email. <br>
<br>
I recently started getting familiar with Kata Containers, as my company<br>
(SUSE) wants to get more involved in this project and is also planning<br>
to integrate Kata in its Containers as a Service platform.<br>
<br>
I'd like to help you out in some way, as learning by doing is effective<br>
and fun :-). So this PR reviewer role and the list of "easy" PR you<br>
proposed seem like a good way to start. <br>
<br>
Cheers,<br>
Marco<br>
<div><div class="h5"><br>
<br>
On Fri, 2018-08-31 at 15:17 +0100, Hunt, James O wrote:<br>
> Hi All,<br>
> <br>
> # TL;DR<br>
> <br>
> If you'd like to get more exposure to Kata, please consider adding<br>
> your name to the PR reviewer rota here:<br>
> <br>
> - <a href="https://github.com/kata-containers/community/wiki/Review-Team-Rota" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community/wiki/<wbr>Review-Team-Rota</a><br>
>   (If you don't have permission to do this, please let us know [1]).<br>
> <br>
> As shown, there are a few gaps which are just waiting for you to put<br>
> your name in! :)<br>
> <br>
> # Myth busting<br>
> <br>
> - You don't need to be a repo maintainer<br>
> <br>
> - You don't have to have done it before :)<br>
> <br>
> - You *can* do it again :)<br>
> <br>
> - There is no limit on the number of folk on the rota for a<br>
> particular week<br>
> <br>
>   If someone already has their name down, you can add yours too! [2].<br>
> <br>
>   Ideally, we'd have lots of folk on the rota every week in all the<br>
> different regions.<br>
> <br>
> - You don't need to have read the PR review guide (although it should<br>
> help)<br>
> <br>
>   It's just a guide and offers some thoughts on things to look out<br>
> for.<br>
> <br>
> - You don't have to spend the entire week doing it<br>
> <br>
>   We appreciate any time you can offer to help clear the PR backlogs.<br>
> <br>
> <br>
> And finally to the biggest myth of them all...<br>
> <br>
> <br>
> <br>
> - You most certainly do *NOT* need to be an expert in golang or Kata<br>
> <br>
>    All contributions are appreciated. Further, we encourage everyone<br>
> to ask as many "silly questions" on PRs as possible. Such "silly<br>
> questions" are invaluable in helping "shake out" what on earth the<br>
> author was thinking at the time they wrote the code (or doc) and can<br>
> provide extremely valuable feedback to them as they may have made<br>
> incorrect assumptions.<br>
> <br>
>   What we're striving for is of course correct code (and doc) but<br>
> also code/doc that looks almost embarrassingly simple. If it looks<br>
> too complex and/or the design/intent is not clear, the PR needs to be<br>
> improved imho. We do this by gently guiding the author to a simpler,<br>
> clearer result by asking questions and making suggestions (politely<br>
> :)<br>
> <br>
> <br>
> # Reasons to help out<br>
> <br>
> - Kudos of course!<br>
> - You'll be helping the project increase velocity (getting all those<br>
> new features and bug fixes landed faster!)<br>
> - Learn more about how Kata works.<br>
> - Learn about our processes.<br>
> - Find areas that interest you.<br>
> <br>
>   There are still a lot of "low-hanging fruit" issues if you're<br>
> looking for an area to start contributing to:<br>
> <br>
>   <br>
> <a href="https://github.com/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Aissue+org%3Akata-containers+label%3A%22good+first+issue%22" rel="noreferrer" target="_blank">https://github.com/issues?<wbr>utf8=%E2%9C%93&q=is%3Aopen+is%<wbr>3Aissue+org%3Akata-containers+<wbr>label%3A%22good+first+issue%22</a><br>
>   <br>
> <a href="https://github.com/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Aissue+org%3Akata-containers+label%3Abitesize" rel="noreferrer" target="_blank">https://github.com/issues?<wbr>utf8=%E2%9C%93&q=is%3Aopen+is%<wbr>3Aissue+org%3Akata-containers+<wbr>label%3Abitesize</a><br>
>   <br>
> <a href="https://github.com/issues?utf8=%E2%9C%93&q=is%3Aopen+is%3Aissue+org%3Akata-containers+label%3A%22help+wanted%22" rel="noreferrer" target="_blank">https://github.com/issues?<wbr>utf8=%E2%9C%93&q=is%3Aopen+is%<wbr>3Aissue+org%3Akata-containers+<wbr>label%3A%22help+wanted%22</a><br>
> <br>
> - Learn how to debug Kata.<br>
> - Learn how our CI works and how to debug issues.<br>
> - ... and did I mention all the Kudos you'll get? :)<br>
> - Kudos<br>
> <br>
> # Further info<br>
> <br>
> - <br>
> <a href="https://github.com/kata-containers/community/blob/master/CODE_OF_CONDUCT.md" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community/blob/<wbr>master/CODE_OF_CONDUCT.md</a><br>
> <br>
>   (This one you do have to read and agree to :)<br>
> <br>
> - <br>
> <a href="https://github.com/kata-containers/community/blob/master/PR-Review-Guide.md" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community/blob/<wbr>master/PR-Review-Guide.md</a><br>
> - <br>
> <a href="https://github.com/kata-containers/community/blob/master/Documentation-Review-Process.md" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community/blob/<wbr>master/Documentation-Review-<wbr>Process.md</a><br>
> - <br>
> <a href="https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community/blob/<wbr>master/CONTRIBUTING.md</a><br>
> <br>
> Cheers,<br>
> <br>
> James<br>
> <br>
> [1] - <a href="https://github.com/kata-containers/community#join-us" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers/community#join-us</a><br>
> [2] - To add multiple names to a github table cell, use "<br/>" like<br>
> this:<br>
> <br>
>        | users1 <br/> user2 <br/> user3 |<br>
> ---<br>
</div></div>> <a href="https://katacontainers.io/o/" rel="noreferrer" target="_blank">https://katacontainers.io/o/</a> | <a href="https://github.com/kata-containers" rel="noreferrer" target="_blank">https://github.com/kata-<wbr>containers</a><br>
<span class="">> Open Source Technology Center<br>
> Intel Corporation (UK) Ltd. - Co. Reg. #1134945 - Pipers Way, Swindon<br>
> SN3 1RJ.<br>
</span>> ______________________________<wbr>_________________<br>
> kata-dev mailing list<br>
> <a href="mailto:kata-dev@lists.katacontainers.io">kata-dev@lists.katacontainers.<wbr>io</a><br>
> <a href="http://lists.katacontainers.io/cgi-bin/mailman/listinfo/kata-dev" rel="noreferrer" target="_blank">http://lists.katacontainers.<wbr>io/cgi-bin/mailman/listinfo/<wbr>kata-dev</a><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><font color="#888888"><span style="font-family:monospace,monospace"><span style="font-family:monospace,monospace">James<br>---</span></span></font></div><div dir="ltr"><a href="https://katacontainers.io/" style="color:rgb(17,85,204);font-family:monospace,monospace" target="_blank">https://katacontainers.io/</a><span style="color:rgb(136,136,136);font-family:monospace,monospace"> | <a href="https://github.com/clearcontainers" target="_blank">https://github.com/kata-containers</a></span><font color="#888888"><span style="font-family:monospace,monospace"><span style="font-family:monospace,monospace"><br></span></span></font></div><div dir="ltr"><font color="#888888"><font face="monospace, monospace">Open Source Technology Center</font><br><font face="monospace, monospace">Intel Corporation (UK) Ltd. - Co. Reg. #1134945 - Pipers Way, Swindon SN3 1RJ.</font></font><br></div></div></div></div></div></div></div></div></div></div></div>
</div>