Check for todo comments as part of running patch build

XMLWordPrintableJSON

    • Fully Compatible
    • v4.9, v4.4
    • DAG 2021-05-17
    • 2
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      We recently added todo comment checking as part of the server commit queue. This is
      useful, but fails a commit-queue entry if something is wrong. It would be nice to have
      some early checks as well to warn engineers that there is an issue before submitting to
      the commit-queue.

      One possible solution is to add a todo check task to required patch builds. This would
      likely rely on the ticket being included as part of tha patch description, but if we
      could find a ticket, check that, otherwise just no-op.


      As a database contributor,
      I want to about unresolved todo in my code before commit-queue time,
      So that I can fix them before I submit my code to the commit-queue.


      AC:

      • A task exists that is part of a required builder that will check for TODO tickets if
        a ticket was provided.

              Assignee:
              David Bradford (Inactive)
              Reporter:
              David Bradford (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: