-
Notifications
You must be signed in to change notification settings - Fork 12.4k
docs: remove stale ERC20VotesComp reference #6182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: typo-fixes
Are you sure you want to change the base?
docs: remove stale ERC20VotesComp reference #6182
Conversation
…elin#6167) Co-authored-by: Luiz Vasconcelos Júnior <64055364+luiz-lvj@users.noreply.github.com> Co-authored-by: ernestognw <ernestognw@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…age (OpenZeppelin#6160) Co-authored-by: Hadrien Croubois <hadrien.croubois@gmail.com>
|
WalkthroughThe governance documentation has been updated to consolidate the ERC20Votes extension description. The entry previously describing two variants—ERC20Votes and ERC20VotesComp—has been renamed and reorganized to present a single unified ERC20Votes extension. The updated documentation now describes core features including support for token supplies exceeding 2^96 and voting power history that can be delegated and queried at past timepoints. References to the Comp variant and its compatibility considerations have been removed. No method signatures or interface changes were introduced. Suggested labels
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
🔇 Additional comments (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The governance guide was still suggesting that ERC20VotesComp is part of the current module set, even though it was removed in v5.0. This commit updates the compatibility section to describe only ERC20Votes as the voting extension used with Governor.