Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add Float64Map #605

Merged
merged 3 commits into from
Mar 24, 2022
Merged

feat: add Float64Map #605

merged 3 commits into from
Mar 24, 2022

Conversation

zuoshuwen
Copy link
Contributor

add Float64Map()

Copy link
Collaborator

@stevenh stevenh left a comment

Choose a reason for hiding this comment

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

Thanks for PR @zuoshuwen just a minor niggle, could you move this new method down to where the other map methods are, say after Int64Map around line 508 so that they are all together :)

@zuoshuwen
Copy link
Contributor Author

Hey @stevenh , thanks for your advice

Copy link
Collaborator

@stevenh stevenh left a comment

Choose a reason for hiding this comment

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

Sorry I missed a typo

redis/reply.go Outdated Show resolved Hide resolved
Co-authored-by: Steven Hartland <steven.hartland@multiplay.co.uk>
@zuoshuwen zuoshuwen requested a review from stevenh March 24, 2022 15:10
@stevenh stevenh merged commit dbebed5 into gomodule:master Mar 24, 2022
@zuoshuwen
Copy link
Contributor Author

Thanks for taking this PR :)

@stevenh
Copy link
Collaborator

stevenh commented Mar 24, 2022

Thank you for the PR 🥇

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants