Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: extend go logger rule #301

Merged
merged 9 commits into from
Feb 22, 2024
Merged

feat: extend go logger rule #301

merged 9 commits into from
Feb 22, 2024

Conversation

elsapet
Copy link
Collaborator

@elsapet elsapet commented Feb 22, 2024

Description

Add more logger methods to existing go_lang_logger rule
Add new go_lang_logger_leak rule (any non-string input)

Checklist

  • I've added a snapshot that shows my rule works as expected.
  • My rule has adequate metadata to explain its use.
  • PR title follows Conventional Commits format

@elsapet elsapet requested a review from cfabianski February 22, 2024 08:51
@elsapet elsapet force-pushed the feat/extend-go-logger-rule branch from ad9585b to fed1552 Compare February 22, 2024 08:52
rules/go/lang/logger_leak.yml Outdated Show resolved Hide resolved
rules/go/lang/logger_leak.yml Outdated Show resolved Hide resolved
rules/go/lang/logger_leak.yml Outdated Show resolved Hide resolved
rules/go/lang/logger_leak.yml Outdated Show resolved Hide resolved
rules/go/lang/logger_leak.yml Outdated Show resolved Hide resolved
@cfabianski cfabianski force-pushed the feat/extend-go-logger-rule branch from 0e7ced8 to 9adfda2 Compare February 22, 2024 13:15
@cfabianski cfabianski merged commit 0fb2db6 into main Feb 22, 2024
20 checks passed
@cfabianski cfabianski deleted the feat/extend-go-logger-rule branch February 22, 2024 13:34
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