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

Integrate alphabetic Clang Tidy check into PR testing

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Open
    • Priority: Minor - P4
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: None
    • Labels:

      Description

      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.

        Attachments

          Activity

            People

            • Assignee:
              backlog-server-storage-engines Backlog - Storage Engines Team
              Reporter:
              alex.cameron Alex Cameron
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: