Add a review policy

[ehuss]

This is an attempt to try to capture what I think should be guidelines for merging and reviewing, based on some of the discussion from the 2024-03-14 meeting.

I placed this within an outline of other team documentation I think we should have to give context to where it fits in. I expect to have these other sections filled in the future.

This explicitly does not cover how "final approval" is given by teams like the lang team, which I think will need a separate process.

Closes #25

[JoelMarcey]

This ironically, is a policy change PR that should be approved by the t-spec team without blocking changes 😊

[JoelMarcey]

@rfcbot poll "t-spec approve"

[JoelMarcey]

Maybe rfcbot doesn't fully work yet.

I was trying to get the t-spec team checkboxes going.

[rfcbot]

Team member @JoelMarcey has asked teams: , for consensus on:

"t-spec approve"

[ehuss]

@JoelMarcey You have to make sure the T-spec label is applied, otherwise rfcbot doesn't know which team to ping. Also, it can take ~10 minutes for it to respond since it is currently polling (once infra sets up the webhook, it should be immediate).

@rfcbot poll "t-spec approve"

[rfcbot]

Team member @ehuss has asked teams: T-spec, for consensus on:

"t-spec approve"

[ehuss]

🤷 Bot now seems confused. Here are checkboxes:

• @JoelMarcey
• @ehuss
• @m-ou-se
• @pnkfelix