Comment/Approve vs Push race

XMLWordPrintable

    • Type: Bug
    • Resolution: Duplicate
    • Priority: Medium
    • None
    • Affects Version/s: 3.5.1
    • Component/s: Pull Requests
    • None
    • Severity 2 - Major
    • 1

      If a reviewer opens a pull-request in browser, then comments or approves it, and meanwhile the pull-request is updated, the comment or approve is applied to the new pull-request state without reviewer noticing it.

      Steps:
      1. Developer creates a pull-request
      2. Reviewer opens the pull-request in browser
      3. developer push another commit C to the pull-request.
      4. reviewer comments a change in the pull-request, which is changed in the commits C

      result: the comment fails to bind to a code
      expected: it should be shown in a context with note "Outdated"

      the approve variation, I think it is serious:
      4a. reviewer approves the pull-request

      result: pull-request is approved without reviewing C
      expected: reviewer must look at the updated PR to be able to approve it. PS: In this case, auto-unapproving plugin was installed, it did not help.

              Assignee:
              Unassigned
              Reporter:
              Maksim Kirillov
              Votes:
              4 Vote for this issue
              Watchers:
              7 Start watching this issue

                Created:
                Updated:
                Resolved: