Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Create ghost inspector subclass #178

Merged
merged 2 commits into from Mar 13, 2014

Conversation

hatched
Copy link
Contributor

@hatched hatched commented Mar 13, 2014

This branch moves a lot of code around and will be cleaned up a lot more when we have the service inspector subclass as well.

To QA
Do extensive exploratory QA on anything that involves the ghost inspector.

Jeff Pihach added 2 commits March 12, 2014 16:54
Created a service inspector utils extension for both the ghost and service inspector subclasses to use.
@jujugui
Copy link
Contributor

jujugui commented Mar 13, 2014

Test FAILed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/550/

@makyo
Copy link
Contributor

makyo commented Mar 13, 2014

QA okay, giving code a thorough look-over.

@makyo
Copy link
Contributor

makyo commented Mar 13, 2014

👍

@hatched
Copy link
Contributor Author

hatched commented Mar 13, 2014

Thanks for the review/qa! :shipit:

@jujugui
Copy link
Contributor

jujugui commented Mar 13, 2014

Build failed: Attempt to land pull request failed
build url: http://ci.jujugui.org:8080/job/juju-gui-merge/185

@jujugui
Copy link
Contributor

jujugui commented Mar 13, 2014

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge

jujugui added a commit that referenced this pull request Mar 13, 2014
This branch moves a lot of code around and will be cleaned up a lot more when we have the service inspector subclass as well.

__To QA__
Do extensive exploratory QA on anything that involves the ghost inspector.
@jujugui jujugui merged commit 1ab00e2 into juju:develop Mar 13, 2014
@hatched hatched deleted the ghost-inspector-subclass branch March 21, 2014 21:05
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.

None yet

3 participants