Skip to content
This repository has been archived by the owner on Mar 25, 2018. It is now read-only.

removed async classes from openstack-glance #53

Merged
merged 1 commit into from
Apr 10, 2013

Conversation

codefromthecrypt
Copy link
Contributor

should serve as a template for how to do the others.

@buildhive
Copy link

Adrian Cole » jclouds-labs #6 SUCCESS
This pull request looks good
(what's this?)

@jclouds
Copy link
Collaborator

jclouds commented Apr 10, 2013

@zack-shoylev wanna glance this over?

On Wednesday, April 10, 2013, BuildHive wrote:

Adrian Cole » jclouds-labs #6https://buildhive.cloudbees.com/job/jclouds/job/jclouds-labs/6/SUCCESS
This pull request looks good
(what's this?) https://www.cloudbees.com/what-is-buildhive


Reply to this email directly or view it on GitHubhttps://github.com/jclouds/jclouds-labs/pull/53#issuecomment-16199490
.

@buildhive
Copy link

Adrian Cole » jclouds-labs #7 SUCCESS
This pull request looks good
(what's this?)

@zack-shoylev
Copy link
Contributor

I especially like the parts where a lot of code is deleted. Looks good...

codefromthecrypt pushed a commit that referenced this pull request Apr 10, 2013
removed async classes from openstack-glance
@codefromthecrypt codefromthecrypt merged commit 476c032 into master Apr 10, 2013
@codefromthecrypt codefromthecrypt deleted the unasync-openstack-glance branch April 10, 2013 20:54
* @see <a href="http://glance.openstack.org/glanceapi.html">api doc</a>
* @see <a
* href="https://github.com/openstack/glance/blob/master/glance/api/v1/images.py">api
* src</a>
Copy link
Member

Choose a reason for hiding this comment

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

We prefer linking to source over documentation?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants