summaryrefslogtreecommitdiff
path: root/pkg/bindings
diff options
context:
space:
mode:
authorEd Santiago <santiago@redhat.com>2022-07-12 10:55:23 -0600
committerMatthew Heon <matthew.heon@pm.me>2022-07-26 13:22:28 -0400
commit52850ce7049277df028fc8839ab99a15be08ead1 (patch)
treee63c505a055b1c92d0995e9ad70488f246d6c9a2 /pkg/bindings
parenta5827e13cd6168741badd29e95a81b3c992bf301 (diff)
downloadpodman-52850ce7049277df028fc8839ab99a15be08ead1.tar.gz
podman-52850ce7049277df028fc8839ab99a15be08ead1.tar.bz2
podman-52850ce7049277df028fc8839ab99a15be08ead1.zip
man page checker: enforce stricter options format
Followup to #14906, in which a nonexistent option was found in a man page. The xref script was designed to catch that, but I was too lax in my parsing: the option was documented using wrong syntax, and the script didn't catch it. Solution: do not allow *any* unrecognized cruft in the option description lines. And fix all improperly-written entries to conform to the rule: **--option**=*value(s)* Two asterisks around option, which must have two dashes. One asterisk around value(s). This is going to cause headaches for some people adding new options, but I don't think I can fix that: there are many factors that make an unparseable line. Adding 'hint' code would make the script even more complex than it is. I have to assume that our contributors are smart enough to look at surrounding context and figure out the right way to specify options. Signed-off-by: Ed Santiago <santiago@redhat.com> <MH: Fixed cherry-pick conflicts> Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'pkg/bindings')
0 files changed, 0 insertions, 0 deletions