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
Avoid uploading key in public repos #1249
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.
LGTM, just a small nit
|
||
addCommand := "add app . --auto-merge=true" | ||
configRepoURL := "ssh://git@gitlab.com/" + gitlabPublicGroup + "/" + configRepoName + ".git" |
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 would suggest to use fmt.Sprintf
to build this string for readability
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 agree, but I decided t keep it this way because that's how URLs are being built across the whole acceptance suite, I'd rather keep consistency over something I prefer.
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 - however, I didn't see any acceptance tests using a public app repo - did I just miss it?
yep, they are there but didn't break because they use a private repo as config url |
bada709
to
1de2f76
Compare
1de2f76
to
dd99c8a
Compare
Closes: #1111
What changed?
Why?
How did you test it?
Release notes
Documentation Changes