Skip to content
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

[Feature][Jetbrains] Support for pushing comments in Codestream to the Merge Request in Gitlab. #174

Open
suguspnk opened this issue Apr 20, 2020 · 3 comments
Labels

Comments

@suguspnk
Copy link

@suguspnk suguspnk commented Apr 20, 2020

It would be good if we have the option to push comments to the merge requests directly from our IDE.
We tried using the Codestream and I think it's really good but due to this, we're not using it for now.

@planteater
Copy link
Collaborator

@planteater planteater commented Apr 20, 2020

@suguspnk - Thanks for the feedback. The details haven't been finalized yet, but in the coming weeks we do plan to make a connection between CodeStream code reviews and GitLab merge requests. I think the general idea is that when a CodeStream code review is approved, you'd then have the option to either link to an existing MR or create a new MR. There would also be an option to have summary information about the code review pushed up into the MR. I'm not sure that we'd push the entire discussion thread into the MR, and more likely would just include a permalink to the code review on CodeStream. But, we'll see what customers want.

@eialanjones
Copy link

@eialanjones eialanjones commented Jun 17, 2020

+1

@planteater
Copy link
Collaborator

@planteater planteater commented Jul 3, 2020

While we don't push the actual comments up to the PR yet, we do now allow you to create a PR from codestream once a code review has been approved. When you do so, summary information about the review, including a permalink, are included in the PR. More details here:

https://www.codestream.com/blog/codestream-8-0-end-to-end-support-for-a-developers-workflow-from-feature-branch-to-pull-request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.