-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Refactor]Refactor config structure #488
[Refactor]Refactor config structure #488
Conversation
Codecov Report
@@ Coverage Diff @@
## master #488 +/- ##
=======================================
Coverage 84.76% 84.76%
=======================================
Files 121 121
Lines 8579 8579
Branches 1416 1416
=======================================
Hits 7272 7272
Misses 953 953
Partials 354 354
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@xvjiarui can you help to review it ? |
kindly ping :p |
0d9dd80
to
bef63b3
Compare
bef63b3
to
a7f2d41
Compare
Refactor config structure
I passed all the unittests and wrote a check function to determine the equivalence between the new config file and the old one