From a2e7306b56f7ffa66efe0434dcddda3259f97efd Mon Sep 17 00:00:00 2001 From: Domen Grabec Date: Tue, 26 Aug 2025 11:47:25 +0200 Subject: [PATCH] add CEI pattern requirement --- templates/Contract-Code-Review.md | 1 + 1 file changed, 1 insertion(+) diff --git a/templates/Contract-Code-Review.md b/templates/Contract-Code-Review.md index f9ec697..ae66d04 100644 --- a/templates/Contract-Code-Review.md +++ b/templates/Contract-Code-Review.md @@ -90,6 +90,7 @@ _What is the PR trying to do? Is this the right thing? Are there bugs in the req - [ ] Oracles can't be bent - [ ] If oracle can be bent, it won't hurt us. - [ ] Do not call balanceOf for external contracts to determine what they will do when they use internal accounting +- [ ] Contract should follow [CEI](https://docs.soliditylang.org/en/latest/security-considerations.html#use-the-checks-effects-interactions-pattern) (Checks - Effects - Interaction) pattern #### Tests