Skip to content

Fix golangci-lint nilnil and revive signals (#526)#526

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

Fix golangci-lint nilnil and revive signals (#526)#526
leoleovich wants to merge 1 commit into
facebook:mainfrom
leoleovich:export-D105831114

Conversation

@leoleovich
Copy link
Copy Markdown
Contributor

@leoleovich leoleovich commented May 20, 2026

Summary:

Fix lint issues triggered by the Go 1.25 bump (D105819402):

  • nilnil: don't return both non-nil value and error
  • revive: remove unnecessary //nolint:revive on ENOENT

Differential Revision: D105831114

@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 20, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 20, 2026

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

@meta-codesync meta-codesync Bot changed the title Fix golangci-lint nilnil and revive signals Fix golangci-lint nilnil and revive signals (#526) May 20, 2026
leoleovich added a commit to leoleovich/time that referenced this pull request May 20, 2026
Summary:

Fix lint issues triggered by the Go 1.25 bump (D105819402):
- nilnil: don't return both non-nil value and error
- revive: remove unnecessary //nolint:revive on ENOENT

Differential Revision: D105831114
@leoleovich leoleovich force-pushed the export-D105831114 branch from 7035658 to 22f6dd5 Compare May 20, 2026 12:32
Summary:

Fix lint issues triggered by the Go 1.25 bump (D105819402):
- nilnil: don't return both non-nil value and error
- revive: remove unnecessary //nolint:revive on ENOENT

Differential Revision: D105831114
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 20, 2026

This pull request has been merged in 23c0765.

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