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: fix typos and formatting #25365
Conversation
Could you please amend your commit to fix the description (you have the description of three different commits in it)? You may drop the As far as I understand, GitHub is having internal issues that would explain most of the CI errors we observe. The Netlify errors are not related to GitHub though:
Can you please look into this? |
Looks like a merge gone wrong -- it should be fixed now. I think GH added the co-authored portion, modified it with the your work email. I ran make docs locally with no errors now. |
fix typos and formatting Co-authored-by: Quentin Monnet <quentin@isovalent.com> Signed-off-by: Peter Jausovec <peter.jausovec@solo.io>
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
@qmonnet what are the next steps for this PR? I see the branch if not up to date with main, but I am hesitant to click merge and get into the same cycle as I did with the previous PR (ie. unsigned commits, unclean commit history etc). Is update with rebase the right option at this point? |
Hi, no need to rebase against Here we're just missing a review from cilium/sig-k8s. @youngnick could you please take a look? |
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.
Sorry about the delay here.
Doc change only, and related tests passed. The full CI suite is not required, we're good to go |
fix typos and formatting in the docs.