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

[chore] do not initialize empty slices #3626

Merged

swiatekm merged 1 commit into open-telemetry:main from atoulme:init_empty_slices

Jan 20, 2025

Contributor

atoulme commented Jan 20, 2025

Apply go best practice to avoid assigning an empty slice.


          [chore] do not initialize empty slices

4b3b63a

atoulme requested a review from a team as a code owner

January 20, 2025 00:07

swiatekm approved these changes

View reviewed changes

Contributor

swiatekm commented Jan 20, 2025

It's a bit annoying there's no linter that can enforce this best practice. Best I found was a recommendation to use ruleguard.

swiatekm merged commit 314df7b into open-telemetry:main

38 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet