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

Fix false positive code detections as described in #4 #9

Merged
merged 8 commits into from
Nov 2, 2021
Merged

Conversation

ntoneee
Copy link
Collaborator

@ntoneee ntoneee commented Nov 1, 2021

Fixes #4

@ntoneee ntoneee requested a review from tm-a-t November 1, 2021 20:46
- binary operations with constants (like "1 - 2", but not "1" or "+1")
  are considered code now
- if a variable which is present in locals() appears in the message, it
  **is** evaluated
Include ctx, msg, print and client to locals to improve code detection
@ntoneee ntoneee requested a review from vanutp November 2, 2021 09:40
@tm-a-t tm-a-t merged commit 04fc88e into master Nov 2, 2021
@ntoneee ntoneee deleted the filter-code branch November 2, 2021 14:10
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.

False positive code detections
2 participants