Feature #469
closed
Hi, it looks interesting.
In the near time, what can be done is:
- to create a tracker called "Review request" with
appropriate workflow which would be used to submit and comment
patches
- add a diff viewer for attachments (the code already exists
for repository diffs), so that attached patches can be easily
reviewed
Then the submitter create a "Review request" issue
and attach its patch to it. Reviewers would be able to view the
diff (just like you see diffs in the repository browser) and
enter their comments by adding notes to the issue.
What do you think ?
- Category set to Issues workflow
Jean-Philippe Lang wrote:
- add a diff viewer for attachments (the code already exists
for repository diffs), so that attached patches can be easily
reviewed
This would be interessting!
- Subject changed from Code Review/Approce workflow? to Code Review/Approve workflow?
Is this issue close as duplicate #2048?
Well I would prefer to close the older one as the newer one has a better history and is more general in naming of the review tools. What do you think?
Both seem to be the same request.
#2048 has many watchers.
I think it is better to close this issue.
- Status changed from New to Closed
- Resolution set to Duplicate
Closed as duplicate of #2048.
Also available in: Atom
PDF