Skip to content

Commit df2665c

Browse files
fix: convert if-else to switch statement in member_ordering.go
Convert if-else chain to expression switch to address QF1003 staticcheck warning. This should resolve the remaining golangci-lint issue.
1 parent d86526e commit df2665c

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

internal/rules/member_ordering/member_ordering.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -857,9 +857,10 @@ func groupMembersByType(members []*ast.Node, memberTypes []interface{}, supports
857857
rankOfCurrentMember := memberRanks[i]
858858
rankOfNextMember := memberRanks[i+1]
859859

860-
if rankOfCurrentMember == previousRank {
860+
switch {
861+
case rankOfCurrentMember == previousRank:
861862
groupedMembers[len(groupedMembers)-1] = append(groupedMembers[len(groupedMembers)-1], member)
862-
} else if rankOfCurrentMember == rankOfNextMember {
863+
case rankOfCurrentMember == rankOfNextMember:
863864
groupedMembers = append(groupedMembers, []*ast.Node{member})
864865
previousRank = rankOfCurrentMember
865866
}

0 commit comments

Comments
 (0)