Skip to content

chore: change interface{} to any#2796

Merged
efectn merged 1 commit intogofiber:mainfrom
nickajacks1:main
Jan 14, 2024
Merged

chore: change interface{} to any#2796
efectn merged 1 commit intogofiber:mainfrom
nickajacks1:main

Conversation

@nickajacks1
Copy link
Member

Description

Fix all occurrences of the use-any lint issue from revive and update the documentation accordingly.

Changes Introduced

List the new features or adjustments introduced in this pull request. Provide details on benchmarks, documentation updates, changelog entries, and if applicable, the migration guide. There is no functional change as any is an alias of interface{}.

  • Documentation Update: Detail the updates made to the documentation and links to the changed files.
  • API Longevity: Discuss the steps taken to ensure that the new or updated APIs are consistent and not prone to breaking changes.

Type of Change

Please delete options that are not relevant.

  • Enhancement (improvement to existing features and functionality)
  • Documentation update (changes to documentation)
  • Code consistency (non-breaking change which improves code reliability and robustness)

Checklist

  • Conducted a self-review of the code and provided comments for complex or critical parts.
  • Updated the documentation in the /docs/ directory for Fiber's documentation.
  • Ensured that new and existing unit tests pass locally with the changes.

@gaby
Copy link
Member

gaby commented Jan 14, 2024

Just to double check, can you run this:

gofmt -r 'interface{} -> any' -w *.go

@nickajacks1
Copy link
Member Author

Just to double check, can you run this:

gofmt -r 'interface{} -> any' -w *.go

Cool, didn't know you could do that.
No further change after running gofmt -r 'interface{} -> any' -w . and no hits from grep -Ri 'interface{}'

Copy link
Member

@gaby gaby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@sixcolors sixcolors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants