Hi Justin, Technically, acceptance into the code base with appropriate review and +1’s signifies lazy consensus of all committers (since any committer can blackball a PR with a single justified -1).
If you (or any community member) is concerned that people didn’t have sufficient opportunity to review, OR passed on the opportunity to review because they didn’t understand the importance of the issue, then it is a good-citizen thing to bring it up in the mailing list -- as you have now done. (You’re a gentleman and a scholar!) If the community has accepted enforcement of coding style via robot, then documenting that behavior seems an appropriate thing to do, certainly not requiring a vote on its own. (Please do!) So the question is, did the community _really_ accept it? I looked over the history of PR#577, and it sure looks to me like the community accepted it with (lengthy) review, due consideration, and two enthusiastic +1’s. As people actually experience the results, there’s always opportunity for second thoughts and fine tuning. I’d say go with the commit, and thanks for the contribution! --Matt On 7/31/17, 12:30 PM, "Justin Leet" <[email protected]> wrote: That PR apparently got merged; I should have noted on the PR that this thread got started. Now there's a new potentially a new question: If that change required a vote, what's the way we want to handle it? Do we revert it or leave it and just follow-on with a PR for any changes we choose to make during discussion / vote? Obviously, this is moot if the answer is we don't need a vote since that PR did have the appropriate +1's. At that point, I'd send an email calling it out and make the appropriate wiki changes for getting people setup. On Mon, Jul 31, 2017 at 1:50 PM, Justin Leet <[email protected]> wrote: > Hi all, > > There was a bit of confusion on code style on at least one ticket, > indirectly related to https://github.com/apache/metron/pull/577. For a > refresher (and also because GitHub is being iffy today), that PR is about > moving to the Google Code Style as discussed in [DISCUSS] Code Style > <https://lists.apache.org/thread.html/bd8afff9ad4fe83feb5e7fdfd5d136947665fd6e4e9856807d06e9a3@%3Cdev.metron.apache.org%3E> > > The feedback in that thread was positive, but no formal vote was done. > I've found at least one vote thread for dev guidelines in the past, but > it's also not part of the bylaws page itself. Does this require just > discussion/community consensus, or does it require (or do we just want) a > vote? And is that in line with what other projects tend to do (if we > choose to fo > > If we're of the opinion that it requires a vote, I'll happy to kick up a > discuss thread with the proposed change, get feedback, and run a vote. It > just delays when the change hits. >
