Remove artifact store interface

[zuzuleinen]

We can remove Store interface

Following reasons:

1. Clean() is implemented only on the local, Done only on the remote
2. ctx was added to the interface only for the remote store purpose
3. In the code they are always used separately so no polymorphism is happening.
4. existence of "not implemented" methods on both implementations

1 & 2 show they really are different behaviours so no benefit of having this interface. Adding new functionalities to one implementation forces us to add params to the second one.

[dketterer]

In the process of removing the shared Store interface it would also be good to remove and avoid the package name store and use more specific names like localartifactstore or remoteartifactstore etc.

[zuzuleinen]

Wanted to implement this in https://github.com/benchkram/bob/pull/227, but didn't get an approval from @Equanox

Will close this