Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-8199

Make s_all accept Python PEP8 compliant line breaks

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: WT10.0.1, 5.0.4, 5.2.0, 5.1.0-rc1, 4.4.11
    • Component/s: None
    • Labels:
      None
    • Story Points:
      3
    • Sprint:
      Storage - Ra 2021-10-18

      Description

      The WiredTiger s_all script incorrectly generates errors (ie false positives) when checking the whitespace (in particular the usage of blank lines) in Python files that follow the industry-wide coding standard for Python (PEP8).

      PEP8 specifies "Surround top-level function and class definitions with two blank lines.".

      If a file contain 2 blank lines, where required by the PEP8 standard, the s_all script rejects that saying that one line is required. This creates a false positive error.

      Definition of done:

      • s_all accepts, without error, Python code that follows the PEP8 standard for blank lines

        Attachments

          Activity

            People

            Assignee:
            jeremy.thorp Jeremy Thorp
            Reporter:
            jeremy.thorp Jeremy Thorp
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: