Skip to content
This repository was archived by the owner on Jul 14, 2021. It is now read-only.

Pull in Chef 15.0.293 and Foodcritic 16 #2072

Merged
merged 4 commits into from
May 14, 2019
Merged

Pull in Chef 15.0.293 and Foodcritic 16 #2072

merged 4 commits into from
May 14, 2019

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented May 14, 2019

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

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"
Copy link
Contributor

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

Copy link
Contributor

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)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor Author

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

tas50 added 2 commits May 14, 2019 11:31
Signed-off-by: Tim Smith <tsmith@chef.io>
Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50 tas50 changed the title Pull in Chef 15.0.293 Pull in Chef 15.0.293 and Foodcritic 16 May 14, 2019
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>
@tas50 tas50 merged commit d105c73 into master May 14, 2019
@chef-ci chef-ci deleted the chef15 branch May 14, 2019 19:30
@lock
Copy link

lock bot commented Jul 13, 2019

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.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants