Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-28465

Refactor eslint.py and clang_format.py

    • Type: Icon: Task Task
    • Resolution: Done
    • Priority: Icon: Major - P3 Major - P3
    • 3.5.5
    • Affects Version/s: None
    • Component/s: None
    • Labels:
      None
    • Fully Compatible
    • Platforms 2017-04-17

      The clang_format.py and eslint.py scripts share a lot of code. These have large amounts of duplicate code including the Repo class for using git, and the parallel_process function.

      This code should be de-duplicated and moved into separate python modules.

      Proposed Layout:
      buildscripts/linters/

      • __init__.py
      • git.py - Repo class and related functions
      • parallel - Parallel Process function

      An example of why code de-duplication is bad is that the Repo class has diverged between these two files, and they now have different features.

            Assignee:
            mark.benvenuto@mongodb.com Mark Benvenuto
            Reporter:
            mark.benvenuto@mongodb.com Mark Benvenuto
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: