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

Fix GitLab integration's multi-line selection. #145

Merged
merged 2 commits into from
Sep 17, 2017

Conversation

AmandaCameron
Copy link
Contributor

It seems GitLab uses a slightly-different syntax for multi-line selection, this fixes the gitlab provider to Do The Right Thing™

@eamodio
Copy link
Member

eamodio commented Sep 16, 2017

@AmandaCameron Thanks for the PR! Given that subtle difference, I think it would probably be best if the GitLabService just doesn't derive from the GitHubService. Would you mind making that happen in this PR (copy/paste of the rest of the functions is perfect fine)?

Also if at all possible could you switch the PR from master to develop?

@AmandaCameron AmandaCameron changed the base branch from master to develop September 16, 2017 23:52
@AmandaCameron
Copy link
Contributor Author

I'll do the code changes tomorrow morning, as I'm getting ready to unwind for the night.

@eamodio
Copy link
Member

eamodio commented Sep 16, 2017

Awesome thank you !

@eamodio eamodio merged commit 04ea3b7 into gitkraken:develop Sep 17, 2017
@eamodio
Copy link
Member

eamodio commented Sep 17, 2017

Thank you for the PR!

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

Successfully merging this pull request may close these issues.

2 participants