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

Make s_all accept Python PEP8 compliant line breaks

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major - P3
    • Resolution: Fixed
    • None
    • WT10.0.1, 5.0.4, 5.2.0, 5.1.0-rc1, 4.4.11
    • None
    • None
    • 3
    • 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

            jeremy.thorp@mongodb.com Jeremy Thorp
            jeremy.thorp@mongodb.com Jeremy Thorp
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: