-
Notifications
You must be signed in to change notification settings - Fork 50
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
Fix LGTM gripes - DO NOT REVIEW #508
Conversation
This pull request fixes 7 alerts when merging 2f4f02b into 01bd930 - view on LGTM.com fixed alerts:
|
This pull request introduces 1 alert and fixes 40 when merging a053459 into 01bd930 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request fixes 55 alerts when merging 036a82d into 01bd930 - view on LGTM.com fixed alerts:
|
This pull request fixes 88 alerts when merging faadc4e into 01bd930 - view on LGTM.com fixed alerts:
|
This pull request fixes 88 alerts when merging a2ab466 into 01bd930 - view on LGTM.com fixed alerts:
|
a2ab466
to
d834284
Compare
This pull request fixes 87 alerts when merging d834284 into d9c3039 - view on LGTM.com fixed alerts:
|
- still lots of warnings to fix
- missed a couple of references to an unused variable I removed
d834284
to
2e1afe2
Compare
Will reopen as new PR with associated JIRA item. |
This pull request fixes 5 alerts when merging 2e1afe2 into 9b1c5ab - view on LGTM.com fixed alerts:
|
No description provided.