Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jakartaee-spec-project-leads] How do I get "Require pull request reviews before merging" removed?



On Fri, 28 Feb 2020 at 18:43, Mark Thomas <markt@xxxxxxxxxx> wrote:
On 28/02/2020 15:21, Kevin Sutter wrote:
> Is removing the "required review" the proper path?

I am a big fan of social controls over technical controls.

I've got no problems with social controls over technical controls.... just so long as the removal of the need for review does not evolve into not even bothering to seek review.   I think the guidelines we worked out for the servlet-api are pretty good and they have no need for technical controls to enforce them (other than perhaps enforcement aids like formatting checks).

I'm shaking the jetty developer tree to see if we can find another set of eyeballs for JSP.   They might not have much time to be proactive, but perhaps we can find somebody to at least help out with review.

cheers






 
--

Back to the top