-
Notifications
You must be signed in to change notification settings - Fork 41
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
feat: Reversion metrics #466
base: master
Are you sure you want to change the base?
Conversation
bsamuels453
commented
Aug 28, 2024
•
edited
Loading
edited
Could we pick one phrase like "Revert Statistics" and use it consistently right now there's three or four ways and it'll be easier to search in the codebase for if we use the same terms? |
agree, I've refactored this to call the component the "reversion reporter" |
@0xalpharush I'm satisfied with how the feature is working; this PR is ready for final review. |