Skip to content
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

#1033 Migrating GEOADD and GEODIST command #1185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

YuvrajGosain
Copy link

@YuvrajGosain YuvrajGosain commented Oct 22, 2024

Issue: #1033

  • Migrated the evalXXX function with the latest definition
  • Update or add unit tests for the new implementation.
  • All unit tests pass successfully.
  • Ensure all integration tests pass successfully.
  • Ensure integration tests are added for the migrated command on multi-threaded resp server.
  • Move Integration tests for the respective commands under the RESP integration tests directory from Async directory
  • Please validate that the documentation for the respective commands is up to date. If not then consider adding them.

@YuvrajGosain YuvrajGosain changed the title Migrating GEOADD and GEODIST command #1033 Migrating GEOADD and GEODIST command Oct 22, 2024
@AshwinKul28 AshwinKul28 added the migration -- command Migrates current eval to a refactored eval for all protocols functionality label Oct 22, 2024
@AshwinKul28 AshwinKul28 linked an issue Oct 24, 2024 that may be closed by this pull request
4 tasks
Copy link
Contributor

@lucifercr07 lucifercr07 left a comment

Choose a reason for hiding this comment

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

@YuvrajGosain can follow sample doc template and update for these commands accordingly it'd be helpful.

@lucifercr07
Copy link
Contributor

@YuvrajGosain please resolve conflicts once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration -- command Migrates current eval to a refactored eval for all protocols functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Command Migration: ('GEOADD', 'GEODIST')
3 participants