<p> Attention is currently required from: Kevin Harwell, Alexander Traud. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17334">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17334?tab=comments">Patch Set #2:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> I don't see the harm in minor breakages to that opinion. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think fundamentally Alexander you have a different view of how the interaction and experience should be for code reviews and the Asterisk project, that differ from everyone else. Your view is much more rigid and you believe that you should only have to do X and the experience should be a certain way. That other people should behave and review as you want, and that Sangoma and other Asterisk members should do things a certain way and do certain things in the project. In reality, it's not that way and I don't think it ever will be.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For everyone else we treat this as a group effort, one of discussion, helping each other out, progressing the project. We aren't as strict, we don't have as defined roles. It's everyone working together.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For code reviews in general the goal of reviewers is to progress the project and get the change into a state where it can be merged. That is something that shouldn't be compromised on, and I truly believe that any and all comments should be posted in a way which is visible to the contributor. Even in the past a minor comment along the lines of "you can do this if you wish" has yielded the discovery of an issue by someone else that does need fixing. The key is reasonable discussion as a result of it. This is not something that should be frowned upon, and it should be expected with every code review. Putting a code review up does not shift full burden to the project and does not mean it will or should be accepted, it's a two way interaction. That being said people will absolutely take over reviews at times if the person is uncomfortable or doesn't have time. There just has to be reasonable communication as such. This is something that contributors have reached out to me privately about, that they were appreciative of the process, the help they received, the comments. They certainly wanted the change to go in quickly but after going through code review that uncovered things they understood why and learned things along the way.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for your 650 projects comment. Undertaking such an action was your choice, a choice that has consequences such as having to interact with the projects. Your choice doesn't mean that the projects have to respond in the way you desire and in my opinion is not a reason to not have discussion regarding changes. In the end everyone is busy, and everyone's time is valuable.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd also like to touch on Gerrit and process briefly. If help is needed or questions about it occur such as the best way to update a review, then they can certainly be brought up on the asterisk-dev mailing list or IRC. We have actively helped people in the last few weeks who had issues or questions.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In regards to using 0 and not marking stuff as -1 I've tried this before. This rarely works, because people don't notice or see the comment so it is generally the equivalent of not commenting at all. A -1 is much more likely to garner attention, and can always be removed by the person who put it there.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for the mentioned wiki page. To the best of my knowledge there has never been a wiki page telling people they can't "argue". There is a code of conduct[1] which is for acceptable behavior, but that doesn't prevent discussion and arguments. </p><p style="white-space: pre-wrap; word-wrap: break-word;">[1] https://wiki.asterisk.org/wiki/display/AST/Asterisk+Community+Code+of+Conduct</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17334">change 17334</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/17334"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4687857b9d56e6f44fd440b73af156691660202e </div>
<div style="display:none"> Gerrit-Change-Number: 17334 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-CC: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Nov 2021 11:17:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Comment-In-Reply-To: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>