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

[LGTM][202106] lgtm to use 202106 branches of swss-common and sairedis #2074

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

stephenxs
Copy link
Collaborator

@stephenxs stephenxs commented Dec 9, 2021

What I did

[LGTM] lgtm to use 202106 branches of swss-common and sairedis

Signed-off-by: stephens stephens@contoso.com

Why I did it

How I verified it

See azure pipelin checkers as a reference:

  • with this commit merged, LGTM passed.
  • without it, LGTM failed as expected, which is why LGTM is marked as failed in the checkers.

Details if related

Signed-off-by: stephens <stephens@contoso.com>
@stephenxs stephenxs changed the title [LGTM] lgtm to use 202106 branches of swss-common and sairedis [LGTM][202106] lgtm to use 202106 branches of swss-common and sairedis Dec 9, 2021
@stephenxs
Copy link
Collaborator Author

With this commit merged, LGTM passed.
Without it, LGTM failed as expected, which is why LGTM is marked as failed in the checkers.

@lguohan lguohan merged commit 38b616b into sonic-net:202106 Dec 9, 2021
@stephenxs stephenxs deleted the 202106-lgtm branch December 9, 2021 08:18
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.

4 participants