add S3Storage

[vincentsarago]

closes #160

To Do

• write tests

cc @ionelmc

[codecov-io]

Codecov Report

Merging #161 into master will decrease coverage by 35.26%.
The diff coverage is 13.95%.

@@             Coverage Diff             @@
##           master     #161       +/-   ##
===========================================
- Coverage    67.8%   32.53%   -35.27%     
===========================================
  Files          16       17        +1     
  Lines        1792     1918      +126     
  Branches      317      340       +23     
===========================================
- Hits         1215      624      -591     
- Misses        497     1261      +764     
+ Partials       80       33       -47

Impacted Files	Coverage Δ
src/pytest_benchmark/storage/s3.py	13% <13%> (ø)
src/pytest_benchmark/utils.py	37.21% <33.33%> (-29.38%)	⬇️
src/pytest_benchmark/table.py	15.18% <0%> (-74.69%)	⬇️
src/pytest_benchmark/fixture.py	15.92% <0%> (-65.93%)	⬇️
src/pytest_benchmark/timers.py	32.25% <0%> (-61.3%)	⬇️
src/pytest_benchmark/histogram.py	26.92% <0%> (-51.93%)	⬇️
src/pytest_benchmark/storage/file.py	25.49% <0%> (-41.18%)	⬇️
src/pytest_benchmark/stats.py	36.2% <0%> (-38.51%)	⬇️
src/pytest_benchmark/session.py	35.22% <0%> (-37.11%)	⬇️
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dafa578...be6e000. Read the comment docs.

[vincentsarago]

@ionelmc Sorry about the failing tests 🤦‍♂ hope the last commit will pass 🤞

[vincentsarago]

Woaa, until yesterday I didn't know about iSort ... and now I understand why the tests were failling. Sorry about this @ionelmc . I hope it will be fine now

[ncclementi]

@vincentsarago Is this something still in the works? We are evaluating using pytest benchmark and having S3 storage would be a great pro.

[vincentsarago]

@ncclementi it worked when I worked on this PR 😄
I haven't used it in a while so if there has been any update on the library this PR may need some update

[ncclementi]

@vincentsarago Yes, I noticed the branch has conflicts, but I'm not sure what's the maintainer's intention in getting this implemented. The PR seems to be very close to done, but it's been a while. Maybe @ionelmc can comment on it?