Labels: Type-Enhancement Priority-Medium
New issue 3663 by shaan.h...@gmail.com: Custom order the diff files
*** For customer support, please post to reviewbo...@googlegroups.com
*** If you have a patch, please submit it to
*** Do not post confidential information in this enhancement request!
What version are you running?
What's the URL of the page this enhancement relates to, if any?
The diff page
Describe the enhancement and the motivation for it.
When there are a lot of files in the diff, it is usually easier to review
if some related files are clustered together. For example, two java classes
that reference each other a lot should be one after the other in the diff
list. Or, you might want to put all test-related files in the end, so that
the code doesn't get cluttered with tests. Hence, it would be great if the
review submitter can order the diff files and all the reviewers see the
files in that particular order.
Taking it further, maybe it is good to group the files into categories, so
that the file list doesn't appear unnecessarily big. For example, I might
have source code changes in 6 files, changes in 6 corresponding test files,
and each test might need 2-3 json files for input data, which will create
18 json files (This has happened). So the crux of the code change (the 6
source files) is lost between the test and test data, and the reviewer
would get an impression that it is too much code, and hence the review gets
What operating system are you using? What browser?
Mac OS X , Chrome
Please provide any additional information below.
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
You received this message because you are subscribed to the Google Groups
To unsubscribe from this group and stop receiving emails from it, send an email
To post to this group, send email to email@example.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.