Uploaded image for project: 'Crucible'
  1. Crucible
  2. CRUC-2998

Iterative pre-commit reviews

    XMLWordPrintable

Details

    • Suggestion
    • Status: Closed (View Workflow)
    • Resolution: Fixed
    • 3.0.0
    • Integrations
    • None
    • Windows XP, Eclipse 3.5
    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

    Description

      On a pre-commit review, we need a way to make it iterative.

      Here is a common scenario:

      1) The developper creates a Pre-commit review
      2) The Reviewer insert a comment in a file to adjust something in the code
      3) The developper gets notified, and makes the correction.

      Now, how this developper sends then update to Crucible so the reviewer can
      get notified and complete the review?

      Right now, we can click the "Add a patch from workspace changes to the Review."

      The problem is that it duplicates the file in the review, wich can become very
      unreadable on multiple files review/multiple comments and changes.

      There should be a way to group the same files together and show only the latest one in the review window, with something like a [+] on left to expand and see other versions.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              83f2395eb3bf Jean-Francois Larouche
              Votes:
              82 Vote for this issue
              Watchers:
              64 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: