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

[console] Display success message after line cleared #1579

Merged
merged 2 commits into from
Apr 25, 2021

Conversation

Blueve
Copy link
Contributor

@Blueve Blueve commented Apr 24, 2021

What I did

Display a success message after line been cleared

How I did it

Simply echo a message after line cleared

How to verify it

Update unit test case

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

admin@sonic:~$ sudo sonic-clear line 1

admin@sonic:~$ 

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

admin@sonic:~$ sudo sonic-clear line 1
Cleared line

admin@sonic:~$ 

@Blueve Blueve requested a review from yxieca April 24, 2021 11:37
@Blueve Blueve merged commit a3d37f1 into sonic-net:master Apr 25, 2021
@Blueve Blueve deleted the dev/jika/clearline branch April 25, 2021 04:08
gitsabari pushed a commit to gitsabari/sonic-utilities that referenced this pull request Jun 15, 2021
* [console] Display success message after line cleared

Signed-off-by: Jing Kan jika@microsoft.com
qiluo-msft pushed a commit that referenced this pull request Aug 3, 2021
* [console] Display success message after line cleared

Signed-off-by: Jing Kan jika@microsoft.com
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 18, 2022
To include following changes:

* d84a8cc 2021-08-05 | [fast-reboot] revert the change of disabling counter polling before fast-reboot (sonic-net#1744) (HEAD -> 202012, github/202012) [Ying Xie]
* e900bc5 2021-08-04 | Add script null_route_helper (sonic-net#1718) [bingwang-ms]
* 85f14e1 2021-08-02 | disk_check updates: (sonic-net#1736) [Renuka Manavalan]
* d68ac1c 2021-05-27 | [console][show] Force refresh all lines status during show line (sonic-net#1641) [Blueve]
* a0e417f 2021-04-25 | [console] Display success message after line cleared (sonic-net#1579) [Blueve]
* 0c6bb27 2021-04-07 | [console] Include Flow Control status in show line result (sonic-net#1549) [Blueve]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants