2016-08-16 22:05:10 +00:00
|
|
|
# Merge Request discussion resolution
|
|
|
|
|
|
|
|
> [Introduced][ce-5022] in GitLab 8.11.
|
|
|
|
|
|
|
|
Discussion resolution helps keep track of progress during code review.
|
|
|
|
Resolving comments prevents you from forgetting to address feedback and lets you
|
|
|
|
hide discussions that are no longer relevant.
|
|
|
|
|
|
|
|
!["A discussion between two people on a piece of code"][discussion-view]
|
|
|
|
|
|
|
|
Comments and discussions can be resolved by anyone with at least Developer
|
|
|
|
access to the project, as well as by the author of the merge request.
|
|
|
|
|
|
|
|
## Marking a comment or discussion as resolved
|
|
|
|
|
|
|
|
You can mark a discussion as resolved by clicking the "Resolve discussion"
|
|
|
|
button at the bottom of the discussion.
|
|
|
|
|
|
|
|
!["Resolve discussion" button][resolve-discussion-button]
|
|
|
|
|
|
|
|
Alternatively, you can mark each comment as resolved individually.
|
|
|
|
|
|
|
|
!["Resolve comment" button][resolve-comment-button]
|
|
|
|
|
|
|
|
## Jumping between unresolved discussions
|
|
|
|
|
|
|
|
When a merge request has a large number of comments it can be difficult to track
|
|
|
|
what remains unresolved. You can jump between unresolved discussions with the
|
|
|
|
Jump button next to the Reply field on a discussion.
|
|
|
|
|
|
|
|
You can also jump to the first unresolved discussion from the button next to the
|
|
|
|
resolved discussions tracker.
|
|
|
|
|
|
|
|
!["3/4 discussions resolved"][discussions-resolved]
|
|
|
|
|
2016-10-31 15:00:42 +00:00
|
|
|
## Only allow merge requests to be merged if all discussions are resolved
|
|
|
|
|
|
|
|
> [Introduced][ce-7125] in GitLab 8.14.
|
|
|
|
|
2016-10-26 21:21:50 +00:00
|
|
|
You can prevent merge requests from being merged until all discussions are
|
|
|
|
resolved.
|
2016-10-31 15:00:42 +00:00
|
|
|
|
|
|
|
Navigate to your project's settings page, select the
|
|
|
|
**Only allow merge requests to be merged if all discussions are resolved** check
|
|
|
|
box and hit **Save** for the changes to take effect.
|
|
|
|
|
|
|
|
![Only allow merge if all the discussions are resolved settings](img/only_allow_merge_if_all_discussions_are_resolved.png)
|
|
|
|
|
|
|
|
From now on, you will not be able to merge from the UI until all discussions
|
|
|
|
are resolved.
|
|
|
|
|
|
|
|
![Only allow merge if all the discussions are resolved message](img/only_allow_merge_if_all_discussions_are_resolved_msg.png)
|
|
|
|
|
2016-12-22 21:54:08 +00:00
|
|
|
## Move all unresolved discussions in a merge request to an issue
|
2016-10-26 21:21:50 +00:00
|
|
|
|
2016-12-22 21:54:08 +00:00
|
|
|
> [Introduced][ce-7180] in GitLab 8.15.
|
2016-10-26 21:21:50 +00:00
|
|
|
|
|
|
|
To delegate unresolved discussions to a new issue you can click the link **open
|
|
|
|
an issue to resolve them later**.
|
|
|
|
|
2016-12-22 21:54:08 +00:00
|
|
|
![Open new issue from unresolved discussions](img/resolve_discussion_open_issue.png)
|
|
|
|
|
2016-10-26 21:21:50 +00:00
|
|
|
This will prepare an issue with content referring to the merge request and
|
|
|
|
discussions.
|
|
|
|
|
|
|
|
![Issue mentioning discussions in a merge request](img/preview_issue_for_discussions.png)
|
|
|
|
|
|
|
|
Hitting **Submit issue** will cause all discussions to be marked as resolved and
|
|
|
|
add a note referring to the newly created issue.
|
|
|
|
|
2016-12-22 21:54:08 +00:00
|
|
|
![Mark discussions as resolved notice](img/resolve_discussion_issue_notice.png)
|
|
|
|
|
2016-10-26 21:21:50 +00:00
|
|
|
You can now proceed to merge the merge request from the UI.
|
|
|
|
|
2016-08-16 22:05:10 +00:00
|
|
|
[ce-5022]: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5022
|
2016-10-31 15:00:42 +00:00
|
|
|
[ce-7125]: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7125
|
2016-10-26 21:21:50 +00:00
|
|
|
[ce-7180]: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7180
|
2016-08-16 22:05:10 +00:00
|
|
|
[resolve-discussion-button]: img/resolve_discussion_button.png
|
|
|
|
[resolve-comment-button]: img/resolve_comment_button.png
|
|
|
|
[discussion-view]: img/discussion_view.png
|
|
|
|
[discussions-resolved]: img/discussions_resolved.png
|