-
Notifications
You must be signed in to change notification settings - Fork 4
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
Migration_and_Updates #527
Conversation
787148c
to
a558033
Compare
[diff-counting] Significant lines: 1414. This diff might be too big! Developer leads are invited to review the code. |
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
0ea0454
to
3b0fa51
Compare
3b0fa51
to
2141559
Compare
I have no idea why the license/cla is not reporting even after signing 😐😐😐 |
Disabled the branch protection rule for license/cla to merge this. I have no idea why it was failing to report that I had already signed |
Summary
This PR is intended to be a complete migration of the codebase from older deprecated libraries to newer versions.
There were many issues with building, testing, etc due to the critical errors that came with the old libraries.
Test Plan
Manual testing was done. Server works fine, installs all packages correctly and runs tests passing.
Notes
Breaking Changes