From 55d69573fc6c9cf2924efeb6265a4f4423a6c17e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ernesto=20Garc=C3=ADa?= Date: Thu, 5 Sep 2024 12:39:42 -0600 Subject: [PATCH] Clarify Governor events' lack of indexed parameters (#5175) Co-authored-by: Hadrien Croubois --- contracts/governance/IGovernor.sol | 3 +++ 1 file changed, 3 insertions(+) diff --git a/contracts/governance/IGovernor.sol b/contracts/governance/IGovernor.sol index a35aa18aba2..67abfdbbae4 100644 --- a/contracts/governance/IGovernor.sol +++ b/contracts/governance/IGovernor.sol @@ -8,6 +8,9 @@ import {IERC6372} from "../interfaces/IERC6372.sol"; /** * @dev Interface of the {Governor} core. + * + * NOTE: Event parameters lack the `indexed` keyword for compatibility with GovernorBravo events. + * Making event parameters `indexed` affects how events are decoded, potentially breaking existing indexers. */ interface IGovernor is IERC165, IERC6372 { enum ProposalState {