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
feat: #132 支持公共库 #133
feat: #132 支持公共库 #133
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.
thanks, 很好的一次提交!
只是现在 Jenkins pipeline 再运行时也会依赖 user和token , 用于jnlp
镜像内的app_checkout.py
,我来单独做一个兼容,更新下jnlp的镜像版本, 然后我们可以合并这个提交。
/LGTM
|
|
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.
/LGTM
} | ||
p.Data["json"] = NewResult(true, rsp, "") | ||
p.ServeJSON() | ||
} | ||
|
||
// 获取仓库域名 | ||
// TODO现在用反射获取,有更好方法再替换 | ||
func getBaseConfigUrl(config interface{}) 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.
将interface{}变更为强类型,或者接口?这样更OO
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.
config是 interface{}
是由 pm.GetIntegrateSettings
返回定义好的
@sampsonye 可以将第1点转为 issues |
Which changes (Bug/Feature):
支持 #132
Special notes for reviewers: