DiceDB
Inconsistent `{storm}`: <Incorrect filepath in DOC.md for storm>
Steps to reproduce
Running the command go run storm/set/main.go will result in an error,. No file called main.go in storm/set folder (line 19 DOC.md)
current path in DOC.md to run storm : go run storm/set/main.go
revised path required to run storm : go run storm/set/set.go
Expected output
The expected path to run storm should be :
go run storm/set/set.go
Observed output
Running storm/set/main.go results :
go: downloading go1.23.0 (linux/amd64)
stat storm/set/main.go: no such file or directory
Expectations for resolution
This issue will be considered resolved when the following things are done
- path is changed from
storm/set/main.gotostorm/set/set.go
You can find the tests under the tests directory of the dice repository and the steps to run are in the README file. Refer to the following links to set up DiceDB and Redis 7.2.5 locally
Is it possible to assign the issue to myself?
EDIT : created branch issue-720, need authorization to push code. Getting an error if done otherwise.
created branch issue-720, need authorization to push code. Getting an error if done otherwise.
@adityakiran1423 you need to fork the repo and make changes in your copy and raise the PR to this repo. Please follow CONTRIBUTING.md for detailed instructions.
@shardul08 , apologies for the context, I meant to say that I am not able to push to my branch (issue-720). I looked up CONTRIBUTING.md, I was going to raise a PR after I pushed to my branch, however I ran into this :
I was wondering if you could help me with this Thanks -Aditya
Seems like you are trying to push the branch to DiceDB repo. You will need to create a fork of DiceDB repo, create a new branch in the forked repo, make changes and push it. Once the new branch is pushed to the forked repo you will be able to raise the PR.
Yep, totally forgot to fork the repo, apologies for the oversight Followed the instructions are raised a PR Link to the PR Thanks for the help -Aditya