-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Gitlab source #184
Gitlab source #184
Conversation
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.
Thannks for the PR!
I'm not sure why CI didn't run, but the next push should kick off a build...
@@ -37,17 +36,16 @@ require ( | |||
github.com/smartystreets/goconvey v0.0.0-20180222194500-ef6db91d284a // indirect | |||
github.com/stretchr/testify v1.3.0 // indirect | |||
github.com/tidwall/pretty v0.0.0-20180105212114-65a9db5fad51 // indirect | |||
github.com/xanzy/go-gitlab v0.15.0 |
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.
Resolve conflicts and regenerate using go mod tidy
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.
I fixed it.
source/gitlab/gitlab.go
Outdated
} | ||
|
||
if u.Host != "" { | ||
err = gn.client.SetBaseURL("https://" + u.Host) |
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.
Use net/url to build the string
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.
I fixed it too.
Pull Request Test Coverage Report for Build 352
💛 - Coveralls |
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.
FYI, if you want the Gitlab source included by default, you'll need to add it to the SOURCE
environment variable in Makefile
and Dockerfile
I add new gitlab source to the project.
I created the pr as you asked @dhui