-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update UG (fix inaccurate commands, improve ordering) #128
Conversation
Codecov ReportBase: 39.13% // Head: 39.56% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #128 +/- ##
============================================
+ Coverage 39.13% 39.56% +0.42%
- Complexity 457 512 +55
============================================
Files 143 147 +4
Lines 2619 2861 +242
Branches 327 371 +44
============================================
+ Hits 1025 1132 +107
- Misses 1529 1654 +125
- Partials 65 75 +10
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
UG Bugs
|
Remaining issues will be addressed in #84 |
Co-authored-by: Devansh Shah <[email protected]>
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.
Preliminary changes LGTM. Approving since the rest of the changes will be made in a separate PR (as discussed internally)
Just an initial PR to improve the UG
Other potential things to do
Fixes
Fixes #168
Fixes #163
Fixes #161
Fixes #158
Fixes #157
Fixes #153
Fixes #152
Fixes #151
Fixes #149
Fixes #148
Fixes #147
Fixes #146
Fixes #144
Fixes #142
Fixes #141
Fixes #137
Fixes #135
Fixes #130
Partially fixes but does not close #132 (confusing error messages)