Jan Damborsky wrote: > Hi Evan, > > > Evan Layton wrote: >> Hi Jan, >> >> The code changes look fine to me and thanks for adding the great comment >> on why the extra blank line is needed here. That definitely makes this >> more readable. :-) > > No problem at all :-) > >> >> I also saw Jack's comment on the use of "CLI cmd:" and wondered about >> that as well. I agree with him that something for example like "exec >> command:" or "command run:" might be more understandable for this prefix. > > Done - the former one was chosen. > I have updated webrev accordingly.
Looks fine to me. > > Thanks a lot for review ! > Jan >
