geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ju@N" <>
Subject Re: [DISCUSS] Blocking merge button in PR
Date Mon, 21 Oct 2019 11:53:01 GMT
+10 to Naba's proposal, it seems the right thing to do and will help us to
prevent accidentally breaking *develop* while keeping focus on people
instead of processes.
I'd add, however, that the *Merge Pull Request* button should remain
disabled until *all CIs have finished*, and only enable it once the *Build,
Unit, Stress Tests and LGTM are green *(that is, force the committer to
wait at least until all CIs are done)*. *I also agree in that that we
should require *at least one* official approval.

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message