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

Cleanup code #1814

Merged
merged 2 commits into from
Jul 5, 2021
Merged

Cleanup code #1814

merged 2 commits into from
Jul 5, 2021

Conversation

kcudnik
Copy link
Contributor

@kcudnik kcudnik commented Jul 1, 2021

What I did
Update gitignore, remove unused variables, fix commandline help message, remove explicit move

Why I did it
Keep code clean

How I verified it
Local run

Details if related

@kcudnik kcudnik requested a review from prsunny as a code owner July 1, 2021 10:18
@kcudnik kcudnik requested a review from qiluo-msft July 1, 2021 10:18
@kcudnik kcudnik requested a review from qiluo-msft July 5, 2021 12:05
@kcudnik kcudnik merged commit 5528ebf into sonic-net:master Jul 5, 2021
@kcudnik kcudnik deleted the cleanup branch July 5, 2021 15:51
judyjoseph pushed a commit that referenced this pull request Aug 10, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
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.

2 participants