-
Notifications
You must be signed in to change notification settings - Fork 269
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
crypto: Rename PreviouslyVerified to VerificationViolation #4067
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Note that previously_verified
returns quite a few matches with a plain grep, so consider whether you want to rename those also or not (and no need to ask for another review from me IMO, unless you're unsure/unclear about one thing in particular).
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4067 +/- ##
==========================================
- Coverage 84.66% 84.64% -0.03%
==========================================
Files 269 269
Lines 28600 28600
==========================================
- Hits 24215 24209 -6
- Misses 4385 4391 +6 ☔ View full report in Codecov by Sentry. |
For consistency with other places, we have now settled on `VerificationViolation` as the best way to express this situation.
a3da650
to
f1ea400
Compare
Part of element-hq/element-meta#2526
For consistency with other places, we have now settled on
VerificationViolation
as the best way to express this situation.