-
-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Describe the bug
It took me way too long to figure out why exclude: tests
didn't work. It seems that counter-intuitively tests
doesn't work to exclude directories when the default target is .
, presumably because Bandit sees the directory as ./tests
and doesn't recognize that tests
and ./tests
refer to the same directory..
To Reproduce
Steps to reproduce the behavior:
- Have a
tests/
subdirectory of files with some bandit issues. - Specify
exclude: tests
- Leave default target as
.
Expected behavior
Bandit action doesn't report warnings for any files under tests/
Actual behavior
Bandit scans the tests and reports warnings.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working