Skip to content

docs: added cpanm installation commands and remove $ sign#6213

Closed
guitu168 wants to merge 8 commits intoapache:masterfrom
guitu168:patch-5
Closed

docs: added cpanm installation commands and remove $ sign#6213
guitu168 wants to merge 8 commits intoapache:masterfrom
guitu168:patch-5

Conversation

@guitu168
Copy link
Contributor

@guitu168 guitu168 commented Jan 26, 2022

What this PR does / why we need it:

Add the cpanm installation command.
Modified etcd startup command to add log output parameters.
Remove the "$" sign before each command.

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

add cpanminus install command,add git install command.
add cpanm installation command, modify etcd start command, remove the "$" sign before the command
@guitu168 guitu168 changed the title docs:improve doc doc:improve docs Jan 26, 2022
@guitu168 guitu168 changed the title doc:improve docs docs:improve doc Jan 26, 2022
@juzhiyuan juzhiyuan changed the title docs:improve doc docs: added cpanm installation commands and remove $ sign Jan 27, 2022
Copy link
Member

@juzhiyuan juzhiyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Contributor

@zaunist zaunist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@guitu168
Copy link
Contributor Author

guitu168 commented Feb 8, 2022

@bisakhmondal hi,can you review this pr? Is there anything wrong with it? thanks.

Copy link
Member

@bisakhmondal bisakhmondal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this. Now the ctrl+c & ctrl+v will be a lot easier : )

@guitu168
Copy link
Contributor Author

guitu168 commented Feb 8, 2022

@bzp2010 hi,can you review this pr ? thanks

@guitu168 guitu168 closed this Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants