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
docs: rephrasing the hubble intro doc #27712
Conversation
@joestringer can you take a look at this when you have time. |
218668b
to
8fed4fd
Compare
@vipul-21 I don't review all PRs, GitHub has assigned CODEOWNERS for the relevant areas to review. As @cilium/tophat for the week I'll trigger CI for you 👍 |
/test |
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 @vipul-21, I left a few suggestions. Could you also put reasoning for the change in the commit message? Based on the linked issue, I understand the change is done to reduce duplication between the two paragraphs.
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.
Can you squash your commits together?
To reduce the duplication between the two paragraphs of hubble intro doc. Co-authored-by: Anna Kapuścińska <ania0102@gmail.com> Signed-off-by: Vipul Singh <vipul21sept@gmail.com>
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!
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
Rephrasing the doc to make sure to avoid duplication
Fixes: #14587