-
Notifications
You must be signed in to change notification settings - Fork 168
Pull in Chef 15.0.293 and Foodcritic 16 #2072
Conversation
This required moving push jobs client to git, which *may* work. We need to push that gem. Signed-off-by: Tim Smith <tsmith@chef.io>
Gemfile
Outdated
@@ -101,7 +101,7 @@ group(:omnibus_package) do | |||
gem "chef-sugar" | |||
gem "mixlib-versioning" | |||
gem "artifactory" | |||
gem "opscode-pushy-client", ">= 2.1" | |||
gem "opscode-pushy-client", git: "https://github.com/chef/opscode-pushy-client.git", branch: "master" |
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.
that'll almost certainly break appbundling
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.
(actually i should stop saying that -- bundle install won't install it correctly into the gem library first)
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.
Switched to the pre gem so we're good to go now
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
We used to dep on this in chef, but it turned out we weren't using it. It can be removed here as well. Signed-off-by: Tim Smith <tsmith@chef.io>
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. |
This required moving push jobs client to git, which may work. We need to push that gem.
Signed-off-by: Tim Smith tsmith@chef.io