Added execution and updated installation section to Readme

[markovamaria]

Related-to:
#105
#104

[markovamaria]

Hi @filipecosta90, could you please take a look? I've added execution section and slightly updated installation section.

By the way, I see 2 installation sections - in start of readme and below (which I updated). Should they be combined to 1?

[codecov-commenter]

Codecov Report

Base: 59.35% // Head: 59.35% // No change to project coverage 👍

Coverage data is based on head (e69c196) compared to base (e5938dd).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   59.35%   59.35%           
=======================================
  Files          24       24           
  Lines        1811     1811           
=======================================
  Hits         1075     1075           
  Misses        736      736           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[slice4e]

Maria, you are right that there are 2 "Installation" section. I would agree that we should probably keep 1 of them. I would say the second one.
Then , notice that there are 2 ways to execute this. We have the coordinator and we have the standalone runner.
I think that you could add your execution section before the section: Testing out redis-benchmarks-spec-sc-coordinator
You could name it: Testing out the redis-benchmarks-spec-runner ...
And explain the difference.

[markovamaria]

Maria, you are right that there are 2 "Installation" section. I would agree that we should probably keep 1 of them. I would say the second one. Then , notice that there are 2 ways to execute this. We have the coordinator and we have the standalone runner. I think that you could add your execution section before the section: Testing out redis-benchmarks-spec-sc-coordinator You could name it: Testing out the redis-benchmarks-spec-runner ... And explain the difference.

Hi @slice4e, I removed one extra Installation section. I moved Execution to Contribution part. As it has installation dependencies, I moved Installation section to Contribution part too.

Honestly I slightly confused that Installation and Execution (testing out) is presented in Contribution part only. What if I don't want to add new benchmarks, but want to see Redis pass rate on benchmarks-specification? O_o
I would suggest to move it or duplicate before "Contributing guidelines". Please let me know what is preferable.

Also I added Navigation in top of page for easy usage.

[markovamaria]

Hi @slice4e , @filipecosta90 , could you please look at CI falures? Seems like there is some connection issue, running tox from Readme locally gives me the same problem. How may I fix it?

[filipecosta90]

rename it to> redis-benchmarks-spec-client-runner

[filipecosta90]

Thank you @markovamaria . Approving!