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

Add ability to create full-context, all-files code review from two URL paths and a repo revision number

XMLWordPrintable

    • Icon: Suggestion Suggestion
    • Resolution: Unresolved
    • None
    • None
    • None
    • 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.

      Similar in purpose to http://tortoisesvn.tigris.org/bigpicture.html#dirdiff

      --------------------------------------------------------
      Example Scenario:

      t0: starting point, project_a is on trunk
      branches/
      trunk/project_a

      t1: bug found, bug fix branch created for project_a
      branches/bug_fix_project_a
      trunk/project_a

      t2: some bug fix changes committed to branches/bug_fix_project_a

      t3: some unrelated changes committed to trunk/project_a

      t4: changes from t3 are merged into branches/bug_fix_project_a

      t5: more bug fix changes committed to branches/bug_fix_project_a

      t6: at this point, branch is ready for review, i want a full-context and all-files code review between head of trunk/project_a and head of branches/bug_fix_project_a
      --------------------------------------------------------

      In the example above, I want to simply tell Crucible "make me a code review which shows all files and full context between http://my_svn_repo/trunk/project_a and http://my_svn_repo/branches/bug_fix_project_a at repo revision x". I wouldn't want the changes from t4 in code review.

      That feature would really help my team since most of our work occurs in "feature/fix" branches and is only merged in after review.

      I realize that this new feature, if accepted, would take some time to complete. In the meantime, can you recommend any tools which would allow me to create a full-context all-files patch file from the input parameters I described?

      Thanks,

      Luke

              Unassigned Unassigned
              72f2352c1721 Luke Patterson
              Votes:
              15 Vote for this issue
              Watchers:
              13 Start watching this issue

                Created:
                Updated: