-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
17: Review comments not reflected on mailing lists #11
Conversation
👋 Welcome back rwestberg! A progress list of the required criteria for merging this PR into |
Webrevs
|
Ehm, small patch 😄 Did you run the integrations tests for GitLab CE and GitHub to test this? |
😊 there were actually a few GitLab CE tests failing, fixed now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
This PR has been reviewed by Erik Helin (ehelin - Reviewer) - changes are approved. |
@rwestberg This change can now be integrated. The commit message will be:
Since the source branch of this PR was last updated there have been 3 commits pushed to the
Since there are no conflicts, your changes will automatically be rebased on top of the above commits when integrating. If you prefer to do this manually, please merge
|
/integrate |
@rwestberg The following commits have been pushed to master since your change was applied: Your commit was automatically rebased without conflicts. |
Hi all,
Please review the following change that adds support for comments attached to reviews. This is not supported by GitLab CE, but is used by GitHub.
Best regards,
Robin
Progress
Approvers