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.

Sign up for GitHub

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

Jump to bottom

[innovium] Fix build break #10838

Open

tonytitus wants to merge 1 commit into sonic-net:master from tonytitus:master

Contributor

tonytitus commented May 15, 2022

Why I did it

Fix build break

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)


          [innovium] Fix build break

tonytitus requested review from sujinmkang, prgeor, vdahiya12 and lguohan as code owners

May 15, 2022 20:33

lgtm-com bot commented May 15, 2022

This pull request introduces 62 alerts when merging 3952655 into 4b581f1 - view on LGTM.com

new alerts:

41 for Wrong number of arguments in a class instantiation
11 for Unused import
4 for Modification of parameter with default
1 for Module imports itself
1 for Unused local variable
1 for Unreachable code
1 for 'import *' may pollute namespace
1 for Variable defined multiple times
1 for Result of integer division may be truncated

tonytitus requested a review from a team as a code owner

June 10, 2022 02:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

sujinmkang Awaiting requested review from sujinmkang

prgeor Awaiting requested review from prgeor

vdahiya12 Awaiting requested review from vdahiya12

lguohan Awaiting requested review from lguohan lguohan is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet