Skip to content

Commit

Permalink
[chore] remove fieldalignment linter TODO (#9174)
Browse files Browse the repository at this point in the history
**Description:** 
Removes a TODO for the fieldalignment linter, offering a reasoning to
keep it disabled moving forward.

**Link to tracking Issue:**
Fixes #2789
  • Loading branch information
atoulme authored Dec 22, 2023
1 parent dbaec76 commit e67cd8b
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,11 @@ linters-settings:
- (github.com/golangci/golangci-lint/pkg/logutils.Log).Fatalf

enable-all: true
# TODO: Enable this and fix the alignment issues.
disable:
# We want to order fields according to readability and grouping them by use cases.
# This linter does not offer a discernible performance improvement as the structs
# defined in this repository are not in the execution hot path.
# See https://github.com/open-telemetry/opentelemetry-collector/issues/2789
- fieldalignment

revive:
Expand Down

0 comments on commit e67cd8b

Please sign in to comment.