-
Notifications
You must be signed in to change notification settings - Fork 62
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: resolves markdown error and misspells in "pushing and pulling artifacts" section #230
Conversation
Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
I made a mistake in the second commit, I misunderstood something, sorry for that, corrected it in the next commit. |
Co-authored-by: Terry Howe <terrylhowe@gmail.com> Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.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.
PTAL
Co-authored-by: Feynman Zhou <feynman@kubesphere.io> Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
noted @FeynmanZhou, thanks for helping. |
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
Co-authored-by: Sajay Antony <sajaya@microsoft.com> Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
Co-authored-by: Asmit Malakannawar <81866614+asmitbm@users.noreply.github.com> Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
Co-authored-by: Asmit Malakannawar <81866614+asmitbm@users.noreply.github.com> Signed-off-by: Shubham Singh <116020663+1Shubham7@users.noreply.github.com>
added the changes @asmitbm. |
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🚀
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
Closes #228
Resolves the 3 errors described in issue #228.