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

LGTM.com - false positive #2227

Open

brettz9 opened this issue Oct 30, 2019 · 1 comment

Labels

false-positive JS

brettz9 commented Oct 30, 2019

Description of the false positive

URL to the alert on the project page on LGTM.com

https://lgtm.com/projects/g/tjunnone/npm-check-updates/snapshot/f5d33ba536cc4e2a248106e7dedf92805511c583/files/lib/version-util.js#xea40c3661a5be0a2:1

As is, this does not need to escape backslashes since the input does not have any.

Thanks!

brettz9 added the false-positive label

max-schaefer added the JS label

Contributor

max-schaefer commented Oct 30, 2019

Thank you for your report, @brettz9! I agree that this is a false positive. We will investigate how to improve our analysis to avoid flagging it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment