Uploaded image for project: 'Skara'
  1. Skara
  2. SKARA-1741

Change the logic of removing CSR label when CSR label is added via '/csr needed' command

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P4 P4
    • 1.0
    • None
    • bots
    • None

      Assuming there is such a situation:

      1. One pr solves issue1 and issue2.
      2. One user thought this pr needs CSR Issue and commented '/csr needed'.
      3. Author of this pr created csr1 of issue1, but then he thought it is not right to create csr of issue1 and withdrawn csr1.
      4. Author of this pr created csr2 of issue2 and csr2 got approved.
      5. In my opinion, the csr label should be removed(But due to SKARA-1729, it is not removed)

      To sum,

      There are two ways that CSR label can be added to a PR
      1. via '/csr needed' command
      2. A CSR issue is discovered

      1. For the first case, if the CSR label is added via '/csr needed', there will be a CSR_NEEDED_MARKER.
      In this case, if we want to remove the csr label. there are two ways
      (1) No active CSR Issue exists and use '/csr unneeded' command
      (2) all of the CSR issues are approved or withdrawn(at least 1 CSR Issue is approved)

      2. For the second case, if the CSR label is added because a CSR Issue is discovered, there will not be CSR_NEEDED_MARKER.
      In this case, if we want to remove the csr label, there is only one way
      (1) all of the CSR issues are approved or withdrawn

            zsong Zhao Song
            zsong Zhao Song
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: