cyberark / conjur-api-dotnet

.NET client for the CyberArk Conjur API
Apache License 2.0
15 stars 11 forks source link

V5support list variable #13

Closed DvirCyberArk closed 6 years ago

DvirCyberArk commented 6 years ago

Hi, We have added list variable API similar to the implementation in V4, this is the first push in a series of upcoming updates to align v5 with v4 functionality

DvirCyberArk commented 6 years ago

Hi a new commit amend all comments was pushed, @dividedmind can you please look and give your opinion again? Thanks!

DvirCyberArk commented 6 years ago

@dividedmind Thank you for writing detailed feedback, don't hesitate to be PITA about nuances we shouldn't compromise on code quality until I catch up with conventions. @dividedmind

DvirCyberArk commented 6 years ago

@dividedmind Any additional comments holding the merge back? thanks

DvirCyberArk commented 6 years ago

Hi @dividedmind, your comment is needed :)