RSB repo commits need approval

[Joel Sherrill]

On Wed, Apr 26, 2023 at 7:06 PM Chris Johns <chrisj at rtems.org> wrote:

> Hi,
>
> All RSB repo commits need to be posted for review and independent approval
> given
> before being pushed to the top level repo.
>

I thought this was the policy for all top level repositories. There is a
degree of trust
on any posted patch that it has been tested by the submitter with the
understanding
that things do slip through. If someone is regularly submitting
modifications without
testing them, then we have a larger problem.

That said, I think we all have accidentally gotten through a reviewed patch
which
broke some odd BSP/architecture or introduced warnings.

This isn't about improving testing. It is just formalizing that a patch
should be
posted for review and an ACK.

--joel


> Thanks
> Chris
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20230427/5668226a/attachment.htm>