From 2aa5bde1f95aa77c579dce7f67d319d110083cbe Mon Sep 17 00:00:00 2001
From: Rick Staa <rick.a.staa@gmail.com>
Date: Mon, 10 Aug 2026 12:52:07 +0200
Subject: [PATCH] docs: prompt for forge tests and scope updates in the PR
 template

`yarn test` is Hardhat only, so nothing currently prompts anyone to run the
forge suite under src/test/. Those are the PoC and Fix pairs proving each past
vulnerability is closed, and they are neither compiled nor run by CI.

Also adds a reminder to update the Immunefi scope page when a deployed address
changes. The L1Migrator redeployment on 2026-06-01 left the scope page pointing
at the superseded contract for ten weeks.

Switches the bold pseudo-headings to real markdown headings while here.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 .github/PULL_REQUEST_TEMPLATE.md | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index fac45509..0d3e1836 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,22 +1,30 @@
-**What does this pull request do? Explain your changes. (required)**
+## What does this pull request do? (required)
+
 <!-- A clear and concise description of what this pull request does. -->
 
-**Specific updates (required)**
-<!--- List out all significant updates your code introduces -->
+## Specific updates (required)
+
+<!-- List out all significant updates your code introduces -->
+
 - ...
 - ...
 - ...
 
-**How did you test each of these updates (required)**
+## How did you test each of these updates? (required)
+
 <!-- A detailed description of how you tested your code changes. Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc. -->
 
+## Does this pull request close any open issues?
 
-**Does this pull request close any open issues?**
 <!-- Fixes # -->
 
+## Checklist
+
+<!-- Go over all the following points, and put an `x` in all the boxes that apply. -->
+<!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
 
-**Checklist:**
-<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
-<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
 - [ ] README and other documentation updated
 - [ ] All tests using `yarn test` pass
+- [ ] `forge test` passes when `contracts/` or `src/test/` changed. Needs an archive
+      Arbitrum RPC; paste the summary line, or say why you could not run it.
+- [ ] If any deployed address changed, the Immunefi scope page is updated