Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-12959

Merge clang-tidy cleanups to jointcal

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Epic Link:
    • Sprint:
      AP S18-1, AP S18-2, AP S18-3
    • Team:
      Alert Production

      Description

      Pim Schellart [X] ran clang-tidy's static code analysis tools on jointcal, and saved the results in a new user branch:

      https://github.com/lsst/jointcal/pull/62

      We should merge these automatic cleanups into jointcal master, by pulling them into a branch for this ticket.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          Parejkoj John Parejko added a comment -

          Pim Schellart [X] I'm ready for you to re-run clang-tidy on jointcal again, but you should wait for Russell Owen's all-clear first.

          Show
          Parejkoj John Parejko added a comment - Pim Schellart [X] I'm ready for you to re-run clang-tidy on jointcal again, but you should wait for Russell Owen 's all-clear first.
          Hide
          Permalink
          pschella Pim Schellart [X] (Inactive) added a comment -

          I have rerun all the checks. Pull request is now on this branch. Please review and merge if you are happy with it.

          Show
          pschella Pim Schellart [X] (Inactive) added a comment - I have rerun all the checks. Pull request is now on this branch. Please review and merge if you are happy with it.
          Hide
          Permalink
          swinbank John Swinbank added a comment -

          Ticket branch is closed, and Pim is no longer with the project. If required, we can re-run clang-tidy on a separate ticket in the future, but this is obsolete.

          Show
          swinbank John Swinbank added a comment - Ticket branch is closed, and Pim is no longer with the project. If required, we can re-run clang-tidy on a separate ticket in the future, but this is obsolete.

            People

            Assignee:
            Parejkoj John Parejko
            Reporter:
            Parejkoj John Parejko
            Watchers:
            John Parejko, John Swinbank, Pim Schellart [X] (Inactive), Russell Owen
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.