-
Notifications
You must be signed in to change notification settings - Fork 412
Fix outdated README sections #1693
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
Fix outdated README sections #1693
Conversation
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.
🚲 🏠 blue!
a5a048e
to
a2d9e59
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.
Nice if we can finally land this one.
Codecov ReportBase: 90.88% // Head: 92.62% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1693 +/- ##
==========================================
+ Coverage 90.88% 92.62% +1.74%
==========================================
Files 85 87 +2
Lines 46220 61297 +15077
Branches 46220 61297 +15077
==========================================
+ Hits 42007 56777 +14770
- Misses 4213 4520 +307
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. |
Thanks for reminding me this PR exists 😅 |
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.
ACK 21a1a45
If we can have another past reviewer looking back.
Feel free to squash, I think. |
21a1a45
to
64183f6
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.
LGTM. Just a few nits.
64183f6
to
e6eb93e
Compare
e6eb93e
to
6817762
Compare
No description provided.