-
Notifications
You must be signed in to change notification settings - Fork 222
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
wip: use web crypto instead of cryptojs #1196
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1196 +/- ##
==========================================
+ Coverage 77.92% 78.00% +0.07%
==========================================
Files 45 45
Lines 1726 1732 +6
Branches 346 346
==========================================
+ Hits 1345 1351 +6
Misses 342 342
Partials 39 39
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
i think now would be a good time to merge this, can you rebase against main, for me the changes look good, i will merge this... |
sorry i didn't realise you had pushed another change, i've rebased again. |
see #909 |
No description provided.