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

[chore]: enable perfsprint linter for receivers #36197

Merged

Conversation

mmorel-35
Copy link
Contributor

Description

perfsprint checks that fmt.Sprintf can be replaced with a faster alternative.

@mmorel-35 mmorel-35 force-pushed the golangci-lint/perfsprint branch 3 times, most recently from f92a078 to 5eee596 Compare November 10, 2024 06:57
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@mmorel-35 mmorel-35 force-pushed the golangci-lint/perfsprint branch from 5eee596 to 356d2c0 Compare November 11, 2024 09:39
@MovieStoreGuy MovieStoreGuy merged commit c92c21e into open-telemetry:main Nov 11, 2024
158 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 11, 2024
@mmorel-35 mmorel-35 deleted the golangci-lint/perfsprint branch November 11, 2024 23:26
pull bot pushed a commit to abaguas/opentelemetry-collector-contrib that referenced this pull request Nov 12, 2024
#### Description

[perfsprint](https://golangci-lint.run/usage/linters/#perfsprint) checks
that fmt.Sprintf can be replaced with a faster alternative.

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
#### Description

[perfsprint](https://golangci-lint.run/usage/linters/#perfsprint) checks
that fmt.Sprintf can be replaced with a faster alternative.

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants