Skip to content

Exclude "/css/" from recursion#1461

Open
shelld3v wants to merge 1 commit intomasterfrom
shelld3v-patch-1
Open

Exclude "/css/" from recursion#1461
shelld3v wants to merge 1 commit intomasterfrom
shelld3v-patch-1

Conversation

@shelld3v
Copy link
Copy Markdown
Collaborator

No description provided.

@maurosoria
Copy link
Copy Markdown
Owner

I dislike this. First, user should know at every moment that "css" directory is being skipped. Let's assume there is an interesting finding in css folder, the tool would miss those findings.

@shelld3v
Copy link
Copy Markdown
Collaborator Author

The chance of having something interesting in the /css/ folder is way too low that it's not worth of the resources and the time we spend on it at all. In case an user actually wants to check that because there is nothing else to check, he can do it manually

@ordinary-hacker
Copy link
Copy Markdown

in my opinion it would be better to keep stuff as is, the user can simply use the --exclude-subdirs flag so I don't find a reason for this

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.

3 participants