Skip to content

Added param usage to README#35

Merged
CarlosLannister merged 1 commit intojunquera:developfrom
MoonsBeast:Fixes
Jul 10, 2025
Merged

Added param usage to README#35
CarlosLannister merged 1 commit intojunquera:developfrom
MoonsBeast:Fixes

Conversation

@MoonsBeast
Copy link
Copy Markdown
Contributor

Added param usage for better undestanding to the user

Added param usage for better undestanding to the user
@CarlosLannister CarlosLannister requested a review from Copilot July 10, 2025 11:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the README by showing the corresponding function parameters for each detection pattern.

  • Replaces generic bullets with explicit flag examples under Credentials & Identity
  • Introduces Location & Organization and Analytics & Tracking sections with parameter usage
  • Extends flag listings for Digital Assets, Social Networks, Cryptographic Hashes, Financial Information, Dark Web URLs, and Paste Sites
Comments suppressed due to low confidence (3)

README.md:54

  • [nitpick] The flag own_name=True may be confusing; consider renaming it to personal_name=True to match the item description and improve clarity.
- **Personal names** (`own_name=True`)

README.md:112

  • [nitpick] All paste-site entries share the same paste=True flag, which could be ambiguous. Consider either using specific flags per domain or clarifying in the text that paste=True covers all listed paste services.
- **justpaste.it** links (`paste=True`)

README.md:49

  • The original list (with -- **Base64** encoded data, etc.) remains above these new entries, resulting in duplicate bullets. Please remove the old generic list items and keep only the new parameterized ones.
- **Base64** encoded data (`base64=True`)

@CarlosLannister CarlosLannister merged commit a234b9d into junquera:develop Jul 10, 2025
2 checks passed
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