Geospatial commands
kamyar1979 opened this issue · 12 comments
Would you add geospatial commands? Can I help?
Happy to review and merge a PR implementing them. Would you be willing to do one?
Sure! I would be happy to contribute!
PLease help me create a good PR! Should I fork the project first?
@kamyar1979 yeah, I see the PR is a bit messed up because you had older fork probably. If your Git skills aren't very high -- I suggest completely deleting your fork first (go to kamyar1979/hedis repo page, and find "delete this repository" in settings), then fork this hedis repo, do your changes, commit and make a PR. This way it will be done on top of the latest commits.
I almost did the job. Still there is some remaining parts in georadius for implementing options, and also tests are not done. Today I will push a primary release.
Pushed first release. Please help mw for adding options, I am confused. I guess I need a instance implementation of encode for GeoRadiusOpts but I am not sure.
Please check my fork and help me make it better! Also I have not yet added tests.
Sorry! I also did some minor fix and housekeeping in your code!
@kamyar1979 where did you push it? You need to close this PR #116 and open a new one which will only have your new commits present
I did it in my fork. I need you to check the code and then let me go on work before asking to merge.
I create a PR from my master to work-in-progress. I cant create a branch. Should I?
@kamyar1979 please make a new branch that starts from the fresh master, once you will do everything correctly you should only see your commits in PR's commit list, not these >100 items seen in #117
Probably need to do something like
git remote add upstream git@github.com:informatikr/hedis.git
git fetch remote
git rebase remote/master
git push -f origin
if you don't feel comfortable doing this – please see my earlier suggestion on completely deleting the repo and re-forking (but be sure to backup the changes to reapply them)
My fork is new! I forked the repo just after your first comment!