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

Add dialects to the reporting rules #475

Merged
merged 8 commits into from
Sep 2, 2021
Merged

Conversation

shaopeng-gh
Copy link
Collaborator

@shaopeng-gh shaopeng-gh commented Sep 2, 2021

also fixed when trying to report on Clang which is not supported because Clang does not include command line in producer flag

@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 1 alert when merging f2f86a6 into c5aeee5 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@eddynaka eddynaka changed the title Binskim - Add dialects to the reporting rules Add dialects to the reporting rules Sep 2, 2021
@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 1 alert when merging f17dccb into 386006b - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 1 alert when merging acf77ee into 386006b - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@eddynaka eddynaka enabled auto-merge (squash) September 2, 2021 13:55
@eddynaka eddynaka merged commit 1609a24 into main Sep 2, 2021
@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 1 alert when merging d3f357b into 386006b - view on LGTM.com

new alerts:

  • 1 for Dereferenced variable may be null

@eddynaka eddynaka deleted the users/shaopeng-gh/reportdialect branch September 2, 2021 14:09
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.

2 participants