On Thu, Mar 24, 2016 at 7:11 AM, Paul Belanger <[email protected]> wrote: > I cannot thing of something, besides linting, to better understand our modules > and sub projects. And interested to here what others think.
I'm certainly glad we have someone who is interested in helping out, I never meant to imply otherwise. Thanks for diving in Andrey :) My fear in a case like this with so many patches from a single new contributor, that it's turned less into a learning exercise and more into a lot of repetition. This quickly becomes much less valuable for getting familiar with things, I'm sure it's very boring, and it is filling up our review queue. I'm also conscious of the fact that not all reordering in Puppet has no impact on functionality, so it does take diligence on the part of the reviewer to make sure we don't break anything in the "harmless" effort to improve style. As for ideas for newcomers, back when we used Launchpad for bug and task tracking we had a low-hanging-fruit tag. This helped me get my first changes in that spanned across a broad number of areas of our infra. These were much more interesting than lint fixes! My hope was that we could somehow continue this in Storyboard. Perhaps even working to take some of the things that established contributors, by reflex, just fix immediately but don't strictly need immediate fixes and tag to give those tasks to newcomers to get their feet wet. I think a separate newcomer thread is in order. -- Elizabeth Krumbach Joseph || Lyz || pleia2 _______________________________________________ OpenStack-Infra mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra
