Remove Storage interface and replace by StorageImpl

[pedro-stanaka]

This is the cleanup I promised, removing the Storage interface to avoid confusion since it will only have one implementation. Moreover, I didnt make the class final, otherwise we won't be able to mock it with prophecy lib.

Feel free to close/ignore if you think this is not needed.

[codecov]

Codecov Report

Merging #366 (ccb12d2) into master (b863acd) will not change coverage.
The diff coverage is 89.09%.

@@            Coverage Diff            @@
##             master     #366   +/-   ##
=========================================
  Coverage     99.35%   99.35%           
  Complexity     1855     1855           
=========================================
  Files           297      297           
  Lines          5858     5858           
=========================================
  Hits           5820     5820           
  Misses           38       38           

Impacted Files	Coverage Δ	Complexity Δ
src/Service/Dist/Storage.php	89.09% <89.09%> (ø)	17.00 <17.00> (?)

---

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 b863acd...ccb12d2. Read the comment docs.

[akondas]

I'm just wondering about the namespace. I really like the changes themselves. What about:

Buddy\Repman\Service\Dist\Storage

(without second Storage)

I apologize in advance for the late pace, but this week will still be quite busy for me 😉

[pedro-stanaka]

No worries, we all have those weeks. I moved the class now, the namespace was strange really.

[akondas]

Thanks @pedro-stanaka 👏