Skip to content
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

[doc]: Update Command-Reference.md #2444

Merged
merged 10 commits into from
Oct 24, 2022

Conversation

mdanish-kh
Copy link
Contributor

@mdanish-kh mdanish-kh commented Oct 17, 2022

What I did

  • Fixed bad hyperlinks from Table of Contents
  • Fixed outdated URLs
  • Formatting fixes
  • Update ZTP commands
  • Update Azure URLs to sonic-net
  • Normalize whitespaces (trim trailing whitespaces)

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

- Fixed bad hyperlinks from Table of Contents
- Fixed outdated URLs
@mdanish-kh mdanish-kh changed the title Update Command-Reference.md [doc]: Update Command-Reference.md Oct 17, 2022
@mdanish-kh
Copy link
Contributor Author

@Blueve - Could you please review?

Copy link
Contributor

@Blueve Blueve 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 contribution the doc! Add a comments, could you check it?

doc/Command-Reference.md Outdated Show resolved Hide resolved
@mdanish-kh mdanish-kh marked this pull request as draft October 19, 2022 14:46
@mdanish-kh mdanish-kh force-pushed the update-command-reference branch 3 times, most recently from bd0f26c to d7f0ccf Compare October 23, 2022 15:14
@mdanish-kh mdanish-kh force-pushed the update-command-reference branch from d7f0ccf to df048bf Compare October 23, 2022 15:17
@mdanish-kh mdanish-kh force-pushed the update-command-reference branch from 5fbb327 to d9154c8 Compare October 23, 2022 15:59
@mdanish-kh mdanish-kh marked this pull request as ready for review October 23, 2022 16:00
@mdanish-kh
Copy link
Contributor Author

Known issue: Kubernetes config commands are missing for which an issue has been created.

@mdanish-kh mdanish-kh requested a review from Blueve October 23, 2022 16:01
@mdanish-kh
Copy link
Contributor Author

@Blueve Could you kindly review again?

@Blueve
Copy link
Contributor

Blueve commented Oct 24, 2022

Thank you so much for the contribution! @mdanish-kh

@Blueve Blueve merged commit c2841b8 into sonic-net:master Oct 24, 2022
@mdanish-kh mdanish-kh deleted the update-command-reference branch October 24, 2022 07:37
@mdanish-kh
Copy link
Contributor Author

Thank you so much for the contribution! @mdanish-kh

You're welcome! As always, thanks for quick review!

dprital added a commit to dprital/sonic-buildimage that referenced this pull request Oct 28, 2022
Update sonic-utilities submodule pointer to include the following:
* 4a3d49d Fix exception in adding mirror_session when gre_type is absent ([sonic-net#2458](sonic-net/sonic-utilities#2458))
* 7e7d05c Update the DBmigrator to support persistent loglevel during warm-upgrade ([sonic-net#2370](sonic-net/sonic-utilities#2370))
* c2841b8 [doc]: Update Command-Reference.md ([sonic-net#2444](sonic-net/sonic-utilities#2444))
* 254cafc Event Counters CLI ([sonic-net#2449](sonic-net/sonic-utilities#2449))
* 2dab0d0 [techsupport] Adding FRR EVPN dumps ([sonic-net#2442](sonic-net/sonic-utilities#2442))
* 3c0aece [show][muxcable] add support for show mux firmware version all ([sonic-net#2441](sonic-net/sonic-utilities#2441))

Signed-off-by: dprital <drorp@nvidia.com>
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
# What I did

Fixed bad hyperlinks from Table of Contents
Fixed outdated URLs
Formatting fixes
Update Azure URLs to sonic-net
Normalize whitespaces (trim trailing whitespaces)
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.

2 participants