-
Notifications
You must be signed in to change notification settings - Fork 9
chore!: remove the cryptobox-migrate feature #942
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: main
Are you sure you want to change the base?
Conversation
0ea8a1a
to
0f562b9
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #942 +/- ##
==========================================
- Coverage 79.27% 79.14% -0.14%
==========================================
Files 113 113
Lines 20853 20713 -140
==========================================
- Hits 16532 16393 -139
+ Misses 4321 4320 -1
Continue to review full report in Codecov by Sentry.
|
0f562b9
to
7349166
Compare
7349166
to
0ac440f
Compare
All clients have moved away from cryptobox so it doesn't make sense to have this.
0ac440f
to
d7d3e6c
Compare
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.
I love getting rid of obsolete features!
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.
Great job that we can get rid of this entirely!
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.
Oops, I misread the GH interface. We're not deleting the file entirely, but still, great to remove this much code from it.
I hadn't previously understood the implications of removing the Cryptobox migration. This code isn't wrong, strictly, but should probably not be merged before August 2025.
All clients have moved away from cryptobox so it doesn't make sense to have this.
📓 This is a breaking change.