Re: Merge protection

+1
________________________________
From: Linda van den Brink <l.vandenbrink@geonovum.nl>
Sent: Monday, September 18, 2023 12:27:14 PM
To: Rob Atkinson <ratkinson@ogc.org>; Grellet Sylvain <S.Grellet@brgm.fr>; Bert Bos <bert@w3.org>
Cc: Simon Cox <dr.shorthair@pm.me>; SDW WG <public-sdw-wg@w3.org>
Subject: RE: Merge protection


This email was received from an external source.   Always check sender details, links & attachments.

I agree too.



When agreed how many positive reviews are required, @Bert Bos<mailto:bert@w3.org> can change the settings for this. I don’t have rights in W3C github repos to do this myself.



Van: Rob Atkinson <ratkinson@ogc.org>
Verzonden: Monday, 18 September 2023 13:07
Aan: Grellet Sylvain <S.Grellet@brgm.fr>
CC: Simon Cox <dr.shorthair@pm.me>; SDW WG <public-sdw-wg@w3.org>
Onderwerp: Re: Merge protection



+1



Rob Atkinson

Senior Research Engineer  | Open Geospatial Consortium (OGC)

Mobile: +61 419 202973

ratkinson@ogc.org<mailto:ratkinson@ogc.org> | ogc.org<http://ogc.org/> | @opengeospatial



[https://portal.ogc.org/files/?artifact_id=99287]

Sign up for OGC News<https://ogc.us4.list-manage.com/subscribe?u=704e02f81107a6caab1568067&id=4e4528fd9d>





On Mon, Sep 18, 2023 at 8:30 PM Grellet Sylvain <S.Grellet@brgm.fr<mailto:S.Grellet@brgm.fr>> wrote:

+1 on all this.



Sylvain



De : Simon Cox <dr.shorthair@pm.me<mailto:dr.shorthair@pm.me>>
Envoyé : lundi 18 septembre 2023 12:27
À : SDW WG <public-sdw-wg@w3.org<mailto:public-sdw-wg@w3.org>>
Objet : Merge protection



For orderly development of the SOSA update, a review or resolution process should be enforced.

I would suggest
- the default branch should be protected
- there should be at least N positive reviews before merging

N should be commensurate with the number of active participants, 2 or 3 to start with.

Cheers - Simon



dr.shorthair@pm.me<mailto:dr.shorthair@pm.me>

+61 403 302 672



On Boonwurrung land


Sent from Proton Mail mobile

[https://portal.ogc.org/files/?artifact_id=99287]

Sign up for OGC News<https://ogc.us4.list-manage.com/subscribe?u=704e02f81107a6caab1568067&id=4e4528fd9d>