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.

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

Adding tg4810m pmon2.0 #36

Open
wants to merge 6 commits into
base: target_master
Choose a base branch
from
Open

Adding tg4810m pmon2.0 #36

wants to merge 6 commits into from

Conversation

lkunjumon
Copy link

- Why I did it

- How I did it

- How to verify it

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

@lgtm-com
Copy link

lgtm-com bot commented Feb 2, 2021

This pull request introduces 38 alerts when merging 238a3ef into ee18483 - view on LGTM.com

new alerts:

  • 15 for Unused local variable
  • 8 for Unused import
  • 6 for Except block handles 'BaseException'
  • 6 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Feb 2, 2021

This pull request introduces 38 alerts when merging e019d49 into ee18483 - view on LGTM.com

new alerts:

  • 15 for Unused local variable
  • 8 for Unused import
  • 6 for Except block handles 'BaseException'
  • 6 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Feb 9, 2021

This pull request introduces 38 alerts when merging 967cde5 into ee18483 - view on LGTM.com

new alerts:

  • 15 for Unused local variable
  • 8 for Unused import
  • 6 for Except block handles 'BaseException'
  • 6 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Mar 5, 2021

This pull request introduces 38 alerts when merging f1b57e6 into ee18483 - view on LGTM.com

new alerts:

  • 15 for Unused local variable
  • 8 for Unused import
  • 6 for Except block handles 'BaseException'
  • 6 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Mar 5, 2021

This pull request introduces 14 alerts when merging 3111350 into ee18483 - view on LGTM.com

new alerts:

  • 6 for Wrong number of arguments in a class instantiation
  • 3 for Unused local variable
  • 3 for 'import *' may pollute namespace
  • 2 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Mar 5, 2021

This pull request introduces 11 alerts when merging f306726 into ee18483 - view on LGTM.com

new alerts:

  • 6 for Wrong number of arguments in a class instantiation
  • 3 for 'import *' may pollute namespace
  • 2 for Unused local variable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lkunjumon