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

Integrate alphabetic Clang Tidy check into PR testing

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: None

      Our most recent pair of interns investigated the use of Clang Tidy to enforce WiredTiger's code style around the alphabetic ordering of variable declarations. This ticket is to track the work required to integrate this as part of our PR testing.

      What needs to be done:

      1. Migrate these changes over to a separate fork of LLVM under the WiredTiger Github org: https://github.com/boboeee/llvm-project/pull/1.
      2. Cleanup the code according to alex.cameron's PR comments.
      3. Build statically linked versions of Clang Tidy for Linux and MacOS and upload them to an S3 bucket.
      4. Integrate them into our dist/ scripts. You can use s_clang_format as a reference for how this should be done.
      5. Add these new scripts to the Evergreen PR testing.

      This task should probably be broken down into separate sub-tickets.

            Assignee:
            backlog-server-storage-engines [DO NOT USE] Backlog - Storage Engines Team
            Reporter:
            alex.cameron@mongodb.com Alex Cameron (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: