| Message ID | 20260901135638.290353-11-jtanenba@redhat.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | ovn-sbctl: Review of ovn-sbctl documentation. | expand |
| Context | Check | Description |
|---|---|---|
| ovsrobot/apply-robot | success | apply and check: success |
| ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
| ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
Hi Jacob, just a small nit on this patch. On Tue, Sep 1, 2026 at 12:02 PM Jacob Tanenbaum via dev <ovs-dev@openvswitch.org> wrote: > > THe command description for lflow-list is missing a capital and definite s/THe/The/ > article. > > "prints numbers of logical flows..." -> "Prints the number of logical > flows..." > > Assisted-by: Claude Opus 4.6, Claude Code > Fixes: 895e02ec0be6 ("ovn-sbctl.c Add logical flows count numbers") > Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com> > --- > utilities/ovn-sbctl.8.xml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/utilities/ovn-sbctl.8.xml b/utilities/ovn-sbctl.8.xml > index 6eb74a4e5..d9e8bca9e 100644 > --- a/utilities/ovn-sbctl.8.xml > +++ b/utilities/ovn-sbctl.8.xml > @@ -434,7 +434,7 @@ > <dd>Alias for <code>lflow-list</code>.</dd> > > <dt><code>count-flows</code> [<var>logical-datapath</var>]</dt> > - <dd>prints numbers of logical flows per table and per datapath.</dd> > + <dd>Prints the number of logical flows per table and per datapath.</dd> > </dl> > > <h2>Remote Connectivity Commands</h2> > -- > 2.55.0 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
diff --git a/utilities/ovn-sbctl.8.xml b/utilities/ovn-sbctl.8.xml index 6eb74a4e5..d9e8bca9e 100644 --- a/utilities/ovn-sbctl.8.xml +++ b/utilities/ovn-sbctl.8.xml @@ -434,7 +434,7 @@ <dd>Alias for <code>lflow-list</code>.</dd> <dt><code>count-flows</code> [<var>logical-datapath</var>]</dt> - <dd>prints numbers of logical flows per table and per datapath.</dd> + <dd>Prints the number of logical flows per table and per datapath.</dd> </dl> <h2>Remote Connectivity Commands</h2>
THe command description for lflow-list is missing a capital and definite article. "prints numbers of logical flows..." -> "Prints the number of logical flows..." Assisted-by: Claude Opus 4.6, Claude Code Fixes: 895e02ec0be6 ("ovn-sbctl.c Add logical flows count numbers") Signed-off-by: Jacob Tanenbaum <jtanenba@redhat.com> --- utilities/ovn-sbctl.8.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)