Kevin Binswanger

Results 16 comments of Kevin Binswanger

This intentionally doesn't attempt to mess with the return type in the hopes of making incremental progress. Of [the four possible solutions](https://github.com/Ecwid/consul-api/issues/49#issuecomment-571794134) I outlined, this is number 1.

The CI failure: > Could not GET 'https://repo.maven.apache.org/maven2/org/junit/jupiter/junit-jupiter-engine/5.1.0/junit-jupiter-engine-5.1.0.pom'. Received status code 403 from server: Forbidden

@vgv Did you have a chance to look at this?

Just curious if you had a chance to look at this or any opinions on the approach.

The problem is that all the different overloaded methods of `deleteKVValue()` return `Response`. In order to indicate whether the delete succeeded or not, we would need to return `Response`. I...

@vgv Do you have a preference for the fix?

How do you feel about the current version?