SecLists icon indicating copy to clipboard operation
SecLists copied to clipboard

Removed comment that causes false positive

Open yechielw opened this issue 2 years ago • 2 comments

yechielw avatar May 04 '22 16:05 yechielw

I don't think the comment should be removed, as it's very helpful. It should be the user's responsability to make sure that comments are ignored in their wordlists

ItsIgnacioPortal avatar May 04 '22 18:05 ItsIgnacioPortal

I do find it odd that the comment is that lower down in the file (unless it relates to the line directly under it - if so, its not clear . I would also suspect it applies to other lines as well).

I would then opt for it to be moved to the start of the file OR put into README.md (or both ;))

g0tmi1k avatar Aug 02 '22 05:08 g0tmi1k

@yechielw are you able to update the pull request?

g0tmi1k avatar Nov 22 '22 11:11 g0tmi1k

Please could this MR be fixed up?

g0tmi1k avatar Mar 09 '23 12:03 g0tmi1k

Closing due to lack of response :(

g0tmi1k avatar May 16 '23 08:05 g0tmi1k