-
Notifications
You must be signed in to change notification settings - Fork 155
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
Adjust the installation command #3119
Conversation
9bb52a1
to
9a3673c
Compare
Signed-off-by: Anish Bista <anishbista053@gmail.com>
9a3673c
to
44adeff
Compare
@PrasadG193 would you mind to take a look here ? |
Can you please share the test plan - how did you test and verified the changes? In this case, can you please attach a screenshot of the rendered page after your changes? |
@anishbista60 I think I wasn't clear with my comment. |
@PrasadG193 Thankyou and sorry previously i was not able to get that. After testing it, looks fine in my local server. |
Thank you, can you please update the Test plan section in the PR description with your testing details? |
@PrasadG193 Done. |
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, thank you for fixing this
@PrasadG193 could you please merge it ? |
Change Overview
Made some changes to installation in
kanister new-docs
Pull request type
Please check the type of change your PR introduces:
Issues
kanister new-docs
#3118 3118Test Plan
docs_new
directory.npm install
.npm install vitepress
npm run docs:dev
and verified that the documentation site loaded successfully without errors and changes are being rendered properly.