Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jsp-dev] Removing restrictions on direct commits

As a first step, I have opened an issue to make the project lead (me) an admin - as I should be according to the Eclipse Handbook. With admin karma, I should then be able to make changes to the review requirements.

Mark

On 03/11/2021 16:33, arjan tijms wrote:
Hi,

Especially for JSP I'm strongly in favour of having the restrictions removed. We don't have an abundance of committers including both very junior and very senior people, and there's practically little to no in-depth reviews because of that.

If there's anything to be corrected it can be done after a commit I'm sure.

So +1 for JSP.

Kind regards,
Arjan



On Tue, Nov 2, 2021 at 5:06 PM Mark Thomas <markt@xxxxxxxxxx <mailto:markt@xxxxxxxxxx>> wrote:

    All,

    In approx 24 hours time I intend to request that the branch
    restrictions
    that prevent committers committing directly to the master branch and
    those that require every PR to be reviewed before merge are removed.

    My reasoning is as follows:
    - I have seen the benefits of these restrictions not being present in EL
    - I'm expecting a number of non-substantive changes will be required to
        successfully complete the release process and PR + review for all of
        them will significantly slow us down
    - Committers are perfectly capable of determining which changes need a
        PR and review and which can be made directly - and if they get it
        wrong changes can easily be reverted

    I was intending to propose this change after the Jakarta 10 release but
    on reflection, I think the sooner, the better.

    Thoughts? Comments? Objects?

    Mark


    _______________________________________________
    jsp-dev mailing list
    jsp-dev@xxxxxxxxxxx <mailto:jsp-dev@xxxxxxxxxxx>
    To unsubscribe from this list, visit
    https://www.eclipse.org/mailman/listinfo/jsp-dev
    <https://www.eclipse.org/mailman/listinfo/jsp-dev>


_______________________________________________
jsp-dev mailing list
jsp-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jsp-dev




Back to the top