Skip to content

Update documentation and code to follow coding convention#733

Merged
Mzack9999 merged 1 commit intoprojectdiscovery:devfrom
mjkim610:style-fix
Sep 15, 2025
Merged

Update documentation and code to follow coding convention#733
Mzack9999 merged 1 commit intoprojectdiscovery:devfrom
mjkim610:style-fix

Conversation

@mjkim610
Copy link
Contributor

ProjectDiscovery code usually follows the rule "Words in names that are initialisms or acronyms (e.g., URL and NATO) should have the same case." (defined in https://google.github.io/styleguide/go/decisions.html#initialisms) SOCKS5 is most commonly written in all-caps, just like HTTP.

This commit updates the documentation and code to all use the same style when refering to SOCKS5 as well as adhere to the coding convention. This commit also updates some other style errors.

Proposed changes

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit/integration/regression tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@mjkim610 mjkim610 changed the title Update documentation and code to follow style guide Update documentation and code to follow coding convention Sep 13, 2025
ProjectDiscovery code usually follows the rule "Words in names
that are initialisms or acronyms (e.g., URL and NATO) should have
the same case." (defined in https://google.github.io/styleguide/
go/decisions.html#initialisms) SOCKS5 is most commonly written in
all-caps, just like HTTP.

This commit updates the documentation and code to all use the same
style when refering to SOCKS5 as well as adhere to the coding
convention. This commit also updates some other style errors.

Signed-off-by: MJ Kim <mjkim610@gmail.com>
@Mzack9999 Mzack9999 merged commit f2381b5 into projectdiscovery:dev Sep 15, 2025
8 checks passed
@mjkim610 mjkim610 deleted the style-fix branch September 15, 2025 21:28
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.

2 participants

Comments