Warn PR authors about the merge policy (#500)

josh-berry · web-flow · commit bae790a6cfe5 · 2024-11-27T13:57:24.000-05:00
**What changed?**

Add a warning to the GitHub PR template reminding everyone that both
Server AND SDK teams must sign off on a PR before it's merged.

**Why?**

'Cause people have merged things a couple times now without SDK review.

**Breaking changes**

None
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,3 +1,5 @@
+_**READ BEFORE MERGING:** All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._
+
 <!-- Describe what has changed in this PR -->
 **What changed?**
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+_READ BEFORE MERGING: All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted._`
	`2`	`+`
`1`	`3`	`<!-- Describe what has changed in this PR -->`
`2`	`4`	`What changed?`
`3`	`5`