Skip to content
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

Update gem dependencies #146

Merged
merged 3 commits into from
Jul 10, 2015

Conversation

andrewjamesbrown
Copy link

This commit simply updates the dependencies for more recent versions.

This commit simply updates the dependencies for more recent versions.
This commit re-adds support for POST /organizations/ORGNAME/users.
Since the code is very similar to
/organizations/ORG/association_requests, we introduced a helper module
to avoid code duplication.
@andrewjamesbrown
Copy link
Author

Hi @marcparadise, is there anything I can do to help move this along? Thanks!


gem 'chef', :github => 'opscode/chef', :tag => '12.1.0.rc.0'
gem 'chef', :github => 'chef/chef', :tag => '12.4.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you update this to 12.4.1 now? 12.4.0 had some regressions.

Copy link
Author

Choose a reason for hiding this comment

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

Done.

@tyler-ball
Copy link
Contributor

👍

@tyler-ball
Copy link
Contributor

Hey @andrewjamesbrown - thanks for this PR. I'm going to merge it into the WIP branch and we can continue discussion in #117

tyler-ball added a commit that referenced this pull request Jul 10, 2015
@tyler-ball tyler-ball merged commit 399c389 into chef:mp/merge-pedants Jul 10, 2015
@andrewjamesbrown andrewjamesbrown deleted the ajb/pr117_fixes branch July 10, 2015 17:20
@jkeiser jkeiser added the bug label Sep 2, 2015
@thommay thommay added Type: Bug Does not work as expected. and removed bug labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants