Code review comment for lp:~mrazik/cupstream2distro-config/jenkins-per-project

Revision history for this message
Francis Ginther (fginther) wrote :

My mistake. I was thinking that 'credentials' was simply the entire contents of the credentials file, but now I see that 'credentials' is just the portion specific to the jenkins_name. So, my suggestion would require more refactoring which isn't really necessary.

Approving as is.

review: Approve

« Back to merge proposal