Skip to content

Fix golangci-lint v2 issues (#525)#525

Closed
leoleovich wants to merge 1 commit into
facebook:mainfrom
leoleovich:export-D105680760
Closed

Fix golangci-lint v2 issues (#525)#525
leoleovich wants to merge 1 commit into
facebook:mainfrom
leoleovich:export-D105680760

Conversation

@leoleovich
Copy link
Copy Markdown
Contributor

@leoleovich leoleovich commented May 19, 2026

Summary:

Fix all 41 pre-existing golangci-lint v2 issues: prealloc capacity hints, lowercase error strings (ST1005), remove redundant embedded field selectors (QF1008), strings.ReplaceAll (QF1004), tagged switch (QF1003), and string conversion (QF1010).

Reviewed By: pmazzini

Differential Revision: D105680760

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 19, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 19, 2026

@leoleovich has exported this pull request. If you are a Meta employee, you can view the originating Diff in D105680760.

@meta-codesync meta-codesync Bot changed the title Fix golangci-lint v2 issues Fix golangci-lint v2 issues (#525) May 19, 2026
leoleovich added a commit to leoleovich/time that referenced this pull request May 19, 2026
Summary:

Fix all 41 pre-existing golangci-lint v2 issues: prealloc capacity hints, lowercase error strings (ST1005), remove redundant embedded field selectors (QF1008), strings.ReplaceAll (QF1004), tagged switch (QF1003), and string conversion (QF1010).

Differential Revision: D105680760
@leoleovich leoleovich force-pushed the export-D105680760 branch from 4977e73 to 1b458c1 Compare May 19, 2026 10:11
Summary:

Fix all 41 pre-existing golangci-lint v2 issues: prealloc capacity hints, lowercase error strings (ST1005), remove redundant embedded field selectors (QF1008), strings.ReplaceAll (QF1004), tagged switch (QF1003), and string conversion (QF1010).

Reviewed By: pmazzini

Differential Revision: D105680760
@leoleovich leoleovich force-pushed the export-D105680760 branch from 1b458c1 to 6406d5d Compare May 19, 2026 10:23
@meta-codesync meta-codesync Bot closed this in ae1fb83 May 19, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 19, 2026

This pull request has been merged in ae1fb83.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant