[Avocado-devel] Review and merge rules

[Plamen Dimitrov]

Hi all,

I am fairly new to the avocado community (about a year or so) and I am not
sure so far I was clear on the reviewing and merging authorization that is
employed in all avocado repositories. I was left with the apparently false
impression that new members of the community can only comment on newer PR-s
and nothing else and merging a PR would require two reviews from authorized
members. However, recently I noticed that we can also review open pull
requests which I didn't know before. Is this true or just wrongly enabled
github switch? If so are there formal rules who can review/merge/comment
anywhere to be found in the official avocado documentation or is it a standard
workflow with the only rules that:

1) two reviews are needed for merge for everyone
2) everyone can review new pull requests

Thanks,
Plamen

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/avocado-devel/attachments/20191204/84c0fa3f/attachment.sig>