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

fix LGTM alerts #232

Merged
merged 1 commit into from
Dec 11, 2020
Merged

fix LGTM alerts #232

merged 1 commit into from
Dec 11, 2020

Conversation

tomato42
Copy link
Member

remove unused imports, bad assert methods, unused variables etc.

@tomato42 tomato42 added the maintenance issues related to making the project usable or testable label Dec 11, 2020
@tomato42 tomato42 self-assigned this Dec 11, 2020
@lgtm-com
Copy link

lgtm-com bot commented Dec 11, 2020

This pull request fixes 3 alerts when merging 375de36 into 24afc18 - view on LGTM.com

fixed alerts:

  • 1 for Comparison using is when operands support __eq__
  • 1 for Explicit export is not defined
  • 1 for Variable defined multiple times

@tomato42
fix LGTM alerts
a690d1f 
remove unused imports, bad assert methods, unused variables etc.
@lgtm-com
Copy link

lgtm-com bot commented Dec 11, 2020

This pull request fixes 3 alerts when merging a690d1f into 24afc18 - view on LGTM.com

fixed alerts:

  • 1 for Comparison using is when operands support __eq__
  • 1 for Explicit export is not defined
  • 1 for Variable defined multiple times

@tomato42 tomato42 merged commit 61133a6 into tlsfuzzer:master Dec 11, 2020
@tomato42 tomato42 deleted the fix-lgtm branch December 11, 2020 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tomato42 tomato42

Labels
maintenance issues related to making the project usable or testable
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@tomato42