-
Notifications
You must be signed in to change notification settings - Fork 170
Fixing bug in expeditor script to update dependencies #1827
Conversation
For loop was using wrong syntax. Also updating dependencies since these need to go in too. Signed-off-by: tyler-ball <tball@chef.io>
@@ -21,7 +21,7 @@ git checkout -b "$branch" | |||
gem install rake | |||
|
|||
tries=6 | |||
for i in {1..$tries}; do | |||
for (( i=1; i<=$tries; i+=1 )); do |
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 guess we can go with a purely bash iterator. 🙄
@@ -203,7 +203,7 @@ GEM | |||
mixlib-log (~> 2.0) | |||
rack (~> 2.0) | |||
uuidtools (~> 2.1) | |||
cheffish (14.0.1) | |||
cheffish (14.0.4) |
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.
Do we want to bring the unrelated cheffish update along in this PR? Or correct #1825 ?
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.
We need to update Inspec too, and this fixes the script that updates the dependencies, so I figured we could do it all here rather than waiting 2+ hours for all the extra builds
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.
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.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
For loop was using wrong syntax. Also updating dependencies since these
need to go in too.
Issues Resolved
☝️
Check List
New functionality includes testsSigned-off-by: tyler-ball tball@chef.io