> On 19 Aug 2015, at 02:52, Pablo Neira Ayuso <[email protected]> wrote:
> 

Hi!

> Just pushed out this to master, including a patch to rename dummy to
> ndummy as you suggested. Will follow up soon by reviewing the other
> concerns you spot during review.

the CI fails because of missing documentation strings in the new command

Documentation error (missing docs): 
<command id='osmux dummy (on|off)'>
        <param name='off' doc='(null)' />

Documentation error (missing docs): 
<command id='osmux dummy (on|off)'>
        <param name='off' doc='(null)' />

with the VTY code every parameter needs to be documented in this case it
needs to be:

        osmux
        dummy
        on
        off

Given the error message I think everything but off is documented.

enjoy the city!

holger

Reply via email to