Feature #469
closed
Code Review/Approve workflow?
0%
Description
Any plans on adding features similar to http://www.review-board.org/ where if enabled, code submitted can be reviewed
and commented with approval or discussion before being committed to the main repository?
Thoughts?
Related issues
Updated by Jean-Philippe Lang almost 18 years ago
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 ?
Updated by Daniel Felix almost 13 years ago
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!
Updated by Daniel Felix almost 13 years ago
- Subject changed from Code Review/Approce workflow? to Code Review/Approve workflow?
Updated by Toshi MARUYAMA over 12 years ago
Is this issue close as duplicate #2048?
Updated by Daniel Felix over 12 years ago
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.
Updated by Toshi MARUYAMA over 12 years ago
#2048 has many watchers.
I think it is better to close this issue.
Updated by Daniel Felix over 12 years ago
- Status changed from New to Closed
- Resolution set to Duplicate
Closed as duplicate of #2048.