Skip to content

Switch to Github API v3. (v2 was turned off on 6/1/2012.)#19

Open
hannahwhy wants to merge 1 commit intowebsterclay:masterfrom
hannahwhy:github-api-v3
Open

Switch to Github API v3. (v2 was turned off on 6/1/2012.)#19
hannahwhy wants to merge 1 commit intowebsterclay:masterfrom
hannahwhy:github-api-v3

Conversation

@hannahwhy
Copy link

This is a quick "works for me" hack. Let me know if there's stylistic guidelines, tests, etc. that you'd like to see.

@tolland
Copy link

tolland commented Jun 15, 2012

works for me. cheers.
chef (0.10.10)
$ ruby -v
ruby 1.9.2p320 (2012-04-20 revision 35421) [x86_64-linux]

@geewiz
Copy link

geewiz commented Jun 30, 2012

Fixed the problem here, too. Chef 10.12.0, ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.3.0]

@nphase
Copy link

nphase commented Jul 2, 2012

Fixed the problem here too, same specs as geewiz. +1

@dfuentes77
Copy link

Manually patching didn't work for me as to this problem. I'm trying to install this cookbook and I keep getting the following error:

$ knife cookbook github install fnichol/chef-rvm/v0.9.0
Installing rvm from git://github.com/fnichol/chef-rvm.git to /Users//chef/.chef/../cookbooks
Checking out the master branch.
Pristine copy branch (chef-vendor-rvm) exists, switching to it.
Removing pre-existing version.
ERROR: The object you are looking for could not be found
Response:

$ gem list chef
chef (0.10.10)
$ knife -v
Chef: 0.10.10
$ ruby -v
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.4.2]

@tjdett tjdett mentioned this pull request Aug 6, 2012
@nphase
Copy link

nphase commented Aug 8, 2012

I've merged pull request #21 (contains this one) into my master (https://github.com/nphase/knife-github-cookbooks) which is what rubygems.org is pointing to as of today. update/cleanup your gem and things should be good to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants