search

web-platform-tests / rfcs

web-platform-tests RFCs
74 stars 63 forks source link

Clarify details around review approvals before merging #199

Open past opened 3 weeks ago

past commented 3 weeks ago

This captures my understanding of the meeting consensus after today's discussion. I didn't try to cover everything that was discussed, just tried to document the policy updates that would avoid a situation like #182 in the future.

We could make more updates, e.g. around what happens if the infra meeting discussion doesn't result in PR comments, what should be the relevant timeout. Another question we might want to tackle is documenting the policy for reverting (#187), but I thought I would start with what I didn't hear a clear objection on.

I'll bring this up in the next WPT infra meeting, too.

past commented 3 weeks ago

Forgot to tag people for comments: @jgraham, @gsnedders, @foolip, @jcscottiii, @web-platform-tests/admins.