Skip to content

rewrite interface{} to any #3386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fukua95
Copy link
Contributor

@fukua95 fukua95 commented May 20, 2025

Hello @ndyakov , Do you agree with replacing interface{} with any in non-constraint scenarios? If approved, we can migrate all interface{} occurrences.
(similar to the commit: golang/go@2580d0e#diff-eca48ce939b837cddc0194885eff25247d2ef2c481b63a667ee14fafcc0f7c7c).

@fukua95 fukua95 requested a review from dmaier-redislabs as a code owner May 20, 2025 11:54
@ndyakov
Copy link
Member

ndyakov commented May 20, 2025

Hello @fukua95, I was considering this and since it will be an API change (although they are the exact same thing), would like to do it just before releasing v10, to not introduce such API (documentation, more or less) change in v9. WDYT?
p.s. and hopefully bump the go version.

@fukua95
Copy link
Contributor Author

fukua95 commented May 20, 2025

Hello @fukua95, I was considering this and since it will be an API change (although they are the exact same thing), would like to do it just before releasing v10, to not introduce such API (documentation, more or less) change in v9. WDYT?

100% agree! This is exactly the right approach.

@ndyakov ndyakov added the v10 label May 20, 2025
@fukua95 fukua95 force-pushed the fmt/interface-to-any branch from e9e10bb to e2b03d6 Compare May 23, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants