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.

Sign up for GitHub

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

Jump to bottom

Remove usage of nolint:gosec added by #10935 #11425

Open
bogdandrutu opened this issue Oct 12, 2024 · 0 comments · May be fixed by #12112
Open

Remove usage of nolint:gosec added by #10935 #11425

bogdandrutu opened this issue Oct 12, 2024 · 0 comments · May be fixed by #12112
Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up

Comments

@bogdandrutu
Copy link
Member

There are lots of // nolint:gosec annotations to move forward with #10935 since the problems are not new. We should fix the wrong conversions where possible.
@bogdandrutu bogdandrutu added the help wanted Good issue for contributors to OpenTelemetry Service to pick up label Oct 12, 2024
@zzzk1 zzzk1 linked a pull request Jan 17, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[chore] Remove unnecessary usage of nolint:gosec zzzk1/opentelemetry-collector
1 participant
@bogdandrutu