Exclude insecure devices on Olm encryption#5457
Conversation
CodSpeed Performance ReportMerging #5457 will not alter performanceComparing Summary
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #5457 +/- ##
========================================
Coverage 88.92% 88.93%
========================================
Files 333 333
Lines 92267 92932 +665
Branches 92267 92932 +665
========================================
+ Hits 82051 82651 +600
- Misses 6374 6427 +53
- Partials 3842 3854 +12 ☔ View full report in Codecov by Sentry. |
(according to the given share strategy)
6b19eae to
cd4b8a7
Compare
|
Code Coverage failure is due to the runner running out of disk space. It succeeded last night. Will re-try in a bit. |
I attempted to free some more space up in #5464. You might need to rebase for this PR to pick up things. |
andybalaam
left a comment
There was a problem hiding this comment.
I got a bit lost in all this and would appreciate you talking me through it if possible.
|
@andybalaam or @poljar Can one of you mash the "merge" button? I don't have permissions to do so. |
|
I'll let the reviewer take car of this, unless Andy doesn't merge by the end of the day. |
Fixes the encrypting part of #4147
Probably easiest to review commit-by-commit
Signed-off-by: