<div dir="ltr">Thanks for this discussion!<div>Yes, let's do this.</div><div>Code reviews also are good because they broaden familiarity with the code across the community.</div><div>Reduces bus factor.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 24, 2016 at 10:43 AM, Harsh Gupta <span dir="ltr"><<a href="mailto:mail@hargup.in" target="_blank">mail@hargup.in</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> So you think no one should make commits, but only open PRs which should<br>
be reviewed?<br>
<br>
</span>+1, we also use this policy in SymPy [1] and I feel that encourages<br>
people to review other's work.<br>
<span class=""><br>
> alias yolo='git commit -am "Deal with it" && git push -f origin master'<br>
<br>
</span>Force push can really mess things up when multiple people are working on<br>
the same repo simultaneously. Say you pulled the master a certain state,<br>
then I pushed some changes to it and then  if you force push without<br>
pulling again, my intermediate commits will be lost. You can the setting<br>
of the repo to prevent force push and I think we should use them. (See<br>
<a href="https://help.github.com/articles/about-protected-branches/" rel="noreferrer" target="_blank">https://help.github.com/<wbr>articles/about-protected-<wbr>branches/</a>)<br>
<br>
The repo can also be configure to make pull requests a requirement to<br>
commit to master.<br>
<br>
[1]: <a href="https://github.com/sympy/sympy/" rel="noreferrer" target="_blank">https://github.com/sympy/<wbr>sympy/</a><br>
<br>
--<br>
  Harsh Gupta<br>
  <a href="mailto:mail@hargup.in">mail@hargup.in</a><br>
<div><div class="h5"><br>
On Mon, Oct 24, 2016, at 09:19 AM, Niels ten Oever wrote:<br>
> Done.<br>
><br>
> So you think no one should make commits, but only open PRs which should<br>
> be reviewed?<br>
><br>
> I think we could give this a try. I am also one of the bad people who<br>
> always pushes to master (alias yolo='git commit -am "Deal with it" &&<br>
> git push -f origin master' ;) )<br>
><br>
><br>
> Cheers,<br>
><br>
> Niels<br>
><br>
> Niels ten Oever<br>
> Head of Digital<br>
><br>
> Article 19<br>
> <a href="http://www.article19.org" rel="noreferrer" target="_blank">www.article19.org</a><br>
><br>
> PGP fingerprint    8D9F C567 BEE4 A431 56C4<br>
>                    678B 08B5 A0F2 636D 68E9<br>
><br>
> On 10/24/2016 10:09 AM, Sebastian Benthall wrote:<br>
> > Hi all,<br>
> ><br>
> > I must ask for one of you to do a code review for me, please.<br>
> ><br>
> > I made a stupid mistake over the weekend while being too hasty to make<br>
> > some changes. I've explained my mistake here:<br>
> ><br>
> > <a href="https://github.com/datactive/bigbang/issues/275" rel="noreferrer" target="_blank">https://github.com/datactive/<wbr>bigbang/issues/275</a><br>
> ><br>
> > I have filed a pull request to fix the mistake I made, but since the<br>
> > real mistake was hastily making a change without asking somebody to<br>
> > check my work, I have to correct that as well.<br>
> ><br>
> > Would one of you please look over this pull request and, if it looks OK<br>
> > to you, merge it?<br>
> ><br>
> > <a href="https://github.com/datactive/bigbang/pull/276" rel="noreferrer" target="_blank">https://github.com/datactive/<wbr>bigbang/pull/276</a><br>
> ><br>
> > Thanks!<br>
> ><br>
> > Seb<br>
> ><br>
> ><br>
> > ______________________________<wbr>_________________<br>
> > Bigbang-dev mailing list<br>
> > <a href="mailto:Bigbang-dev@data-activism.net">Bigbang-dev@data-activism.net</a><br>
> > <a href="https://lists.ghserv.net/mailman/listinfo/bigbang-dev" rel="noreferrer" target="_blank">https://lists.ghserv.net/<wbr>mailman/listinfo/bigbang-dev</a><br>
> ><br>
><br>
> ______________________________<wbr>_________________<br>
> Bigbang-dev mailing list<br>
> <a href="mailto:Bigbang-dev@data-activism.net">Bigbang-dev@data-activism.net</a><br>
> <a href="https://lists.ghserv.net/mailman/listinfo/bigbang-dev" rel="noreferrer" target="_blank">https://lists.ghserv.net/<wbr>mailman/listinfo/bigbang-dev</a><br>
</div></div>> Email had 1 attachment:<br>
> + signature.asc<br>
>   1k (application/pgp-signature)<br>
<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
Bigbang-dev mailing list<br>
<a href="mailto:Bigbang-dev@data-activism.net">Bigbang-dev@data-activism.net</a><br>
<a href="https://lists.ghserv.net/mailman/listinfo/bigbang-dev" rel="noreferrer" target="_blank">https://lists.ghserv.net/<wbr>mailman/listinfo/bigbang-dev</a><br>
</div></div></blockquote></div><br></div>