-
Notifications
You must be signed in to change notification settings - Fork 282
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: broken links and typos #4117
Conversation
Signed-off-by: Sachin Sahu <[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.
Thanks for fixing the documentation and spell checks. Some minor nits. Other wise looks good.
Codecov Report
@@ Coverage Diff @@
## main #4117 +/- ##
==========================================
- Coverage 93.27% 92.99% -0.28%
==========================================
Files 187 187
Lines 5753 5753
==========================================
- Hits 5366 5350 -16
- Misses 387 403 +16 |
Signed-off-by: Rishabh Singh <[email protected]> Signed-off-by: Sachin Sahu <[email protected]>
Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: Sachin Sahu <[email protected]>
Signed-off-by: Rishabh Singh <[email protected]> Signed-off-by: Sachin Sahu <[email protected]>
Signed-off-by: Rishabh Singh <[email protected]> Signed-off-by: Sachin Sahu <[email protected]>
Signed-off-by: Sachin Sahu <[email protected]> Signed-off-by: Sachin Sahu <[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.
LGTM! Thanks!
Hi @SachinSahu431 I am merging this for now. You can try rebasing the git commits squash to 1 from next time. Makes it cleaner. |
Description
Fix broken links and typographical errors in the existing readme files.
For example this section refers to opensearch-2.8.0-test.yml file, which is no longer available on main branch, and has been moved to opensearch-2.8.0 branch.
There are also some other changes such as removing clearly noticeable typos, unclosed brackets, adding missing link to Slack public release channel, etc.
Issues Resolved
None
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.