Is removing the "required
review" the proper path? It seems that we have a bigger issue
with JSP (and possibly other components) if we are not getting the proper
and needed participation. If the current committers on JSP are not
helping out, then maybe we need some fresh participants? Are there
others that should be nominated for committership? Or, do we need
to "wake up" some of the current participants?
Mark, I'm hitting
a similar issue with the JSTL project with no reviews happening on one
of my PRs. But, I'm not even a committer, so even removing that review
requirement won't help me.
My thought is
that we should try to address the problem of inactivity rather than lifting
the required review. Otherwise, we could end up with individuals
taking a project down a path that is not good for the long-term prospects
of the project. Not implying this about your participation, Mark.
I'm just using this as an example.
---------------------------------------------------
Kevin Sutter
STSM, MicroProfile and Jakarta EE architect @ IBM
e-mail: sutter@xxxxxxxxxx Twitter: @kwsutter
phone: tl-553-3620 (office), 507-253-3620 (office)
LinkedIn: https://www.linkedin.com/in/kevinwsutter
From:
Ivar
Grimstad <ivar.grimstad@xxxxxxxxxxxxxxxxxxxxxx>
To:
JakartaEE
Spec Project Leadership discussions <jakartaee-spec-project-leads@xxxxxxxxxxx>
Date:
02/28/2020
07:25
Subject:
[EXTERNAL]
Re: [jakartaee-spec-project-leads] How do I get "Require pull request
reviews before merging" removed?
Sent
by: jakartaee-spec-project-leads-bounces@xxxxxxxxxxx
Hi Mark,
No need for a formal vote. And email
on the project mailing list should be sufficient.
First of all, encourage committers to
review the PR.
Then suggest the change to lift the review
requirement. If nobody opposes, approve by lazy consensus and file the
Bugzilla for it.
Ivar
On Fri, Feb 28, 2020 at 6:22 AM arjan
tijms <arjan.tijms@xxxxxxxxx>
wrote:
Hi,
I'm in favour of removing that requirement
for JSP and at least for Mark. There's a setting for that on GitHub that
admins can simply switch off. Not sure if it can be done per person or
per group.
We have version control so anything controversial
can be rolled back. I'm not expecting Mark to just do anything controversial
out of the blue. Indeed, IMHO reviews for the kind of work we're currently
doing on projects that are not that active are mostly counter productive.
Kind regards,
Arjan
On Fri, Feb 28, 2020 at 12:16 PM Jonathan
Gallimore <jgallimore@xxxxxxxxxxxxx>
wrote:
Hi Mark,
I'm assuming this is the PR you're referring
to: https://github.com/eclipse-ee4j/jsp-api/pull/139
> Currently at 19 days and counting.
That does sound like a long time. Do
we need more committers on the project? Is there a way that we can give
people review access, but perhaps not commit?
I'm not a committer on the JSP API project,
so a review from me wouldn't count towards actually getting it merged,
but I'd be very happy and willing to review your PR and provide feedback.
I can't do it immediately, would be able to look at it this afternoon/evening,
so you could expect something by Monday morning.
Thanks for flagging this up - it certainly
sounds like we need to address it quickly - one way or another.
Jon
On Fri, Feb 28, 2020 at 10:59 AM Mark
Thomas <markt@xxxxxxxxxx>
wrote:Hi,
My work on the JSP project is stalled - yet again - waiting for a PR to
be approved. This time it is for the first pass at tidying up the spec
doc. Currently at 19 days and counting.
I would like to get the "Require pull request reviews before merging"
setting removed from the project. How do I go about doing that? Is it a
request via Bugzilla? Do I need to hold a project vote first (not sure
who is actually going to vote if I did)? Something else?
Thanks,
Mark
_______________________________________________
jakartaee-spec-project-leads mailing list
jakartaee-spec-project-leads@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://www.eclipse.org/mailman/listinfo/jakartaee-spec-project-leads
--
Jonathan Gallimore
http://www.tomitribe.com_______________________________________________
jakartaee-spec-project-leads mailing list
jakartaee-spec-project-leads@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://www.eclipse.org/mailman/listinfo/jakartaee-spec-project-leads
_______________________________________________
jakartaee-spec-project-leads mailing list
jakartaee-spec-project-leads@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://www.eclipse.org/mailman/listinfo/jakartaee-spec-project-leads
-- Ivar
Grimstad
Jakarta
EE Developer Advocate | Eclipse
Foundation, Inc.
Eclipse
Foundation:
The Platform for Open Innovation and Collaboration_______________________________________________
jakartaee-spec-project-leads mailing list
jakartaee-spec-project-leads@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe
from this list, visit
https://www.eclipse.org/mailman/listinfo/jakartaee-spec-project-leads
_______________________________________________
jakartaee-spec-project-leads mailing list
jakartaee-spec-project-leads@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://www.eclipse.org/mailman/listinfo/jakartaee-spec-project-leads