[CHANGED] Listing Key Value stores and object stores to return status instead of interface

[piotrpio]

This PR addresses nats-io/nats-architecture-and-design#154

This PR changes the signatures of KeyValueStores() and ObjectStores() methods. For KV, this is not a breaking change since this functionality is yet to be released. For object store, this is a breaking change since these methods were released as part of v1.17.0.

[coveralls]

Coverage decreased (-0.07%) to 85.875% when pulling e6fe8b5 on change-kv-and-object-list into 1ee76ea on main.

[kozlovic]

Ok with the change, but since this is a breaking change, I would recommend that we either release v1.18.0 "soon", or we start to post PR against a "dev" branch as opposed to main, so that we keep breaking changes out of main in case we need to provide a patch to existing version before being able to release the next minor release.

I don't think that there is anything wrong releasing v1.18.0 "just" after v1.17.0, especially that most of commits on main right now are [ADDED] or [CHANGED]. So if you think that the next release is going to be v1.18.0 (and not 1.17.1), then go ahead and merge. If not, let's create the "dev" branch and switch this PR against that branch.

[piotrpio]

@kozlovic Got it, I think it makes sense to do the v1.18.0 release after merging this PR. What do you think about introducing a breaking change in object store's signature (that has already been released).

[ripienaar]

Object store is still experimental so I think it’s fine