-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
*: fix more typos #7619
*: fix more typos #7619
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7619 +/- ##
=======================================
Coverage 82.01% 82.02%
=======================================
Files 361 361
Lines 27813 27890 +77
=======================================
+ Hits 22812 22877 +65
- Misses 3822 3826 +4
- Partials 1179 1187 +8
|
thanks @NathanBaulch. Could you add the release notes? |
LGTM. Adding @arvindbr8 for second review |
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.
Thanks for this. Looks like a very detailed review of our documentation.
I can't seem to get Mergeable to run again. Trying a re-trigger by closing and reopening this PR |
Another batch of typos similar to #7487, plus a few minor grammar corrections.
FWIW, top 10 corrections are:
RELEASE NOTES: none