Uploaded image for project: 'Fast Datapath Product'
  1. Fast Datapath Product
  2. FDP-1127

Add spellcheck support for Python files in OVS's checkpatch.

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • None
    • openvswitch3.5
    • 3
    • False
    • Hide

      None

      Show
      None
    • False
    • Hide

      Given that the OVS checkpatch script is used for reviewing code,

      When a Python file is submitted for review,

      Then the script should check for spelling errors in comments and docstrings and flag any issues as warnings without blocking the commit process.

      Show
      Given that the OVS checkpatch script is used for reviewing code, When a Python file is submitted for review, Then the script should check for spelling errors in comments and docstrings and flag any issues as warnings without blocking the commit process.
    • rhel-9
    • rhel-sst-network-fastdatapath-ovsdpdk
    • ssg_networking

      The OVS checkpatch script currently only checks C file comments for spelling issues. We would like to extend this to also check Python code.

      Maybe this can also be done as part of the flake8 test; however, those are currently hard errors, which might not work well. If they were warnings, like with checkpatch, they might be missed during code review or maintainer merges.

       

       

       

              ovsdpdk-triage ovsdpdk triage
              echaudro@redhat.com Eelco Chaudron
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: