/juju-gui

Remove editorial.js and charmresults.js #704

Merged
merged 2 commits into from Mar 10, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@hatched @jujugui @kadams54
@hatched
Owner

hatched commented Mar 10, 2015

These two files are unused, left over from the restructuring of the charm list.
@hatched
Owner

hatched commented Mar 10, 2015

To QA

Play around with the charmbrowser, it should work as usual.
@jujugui
Member

jujugui commented Mar 10, 2015

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://ci.jujugui.org:8080//job/juju-gui/2402/
Test FAILed.
@kadams54
Contributor

kadams54 commented Mar 10, 2015

👍 QA is 🆗, great to see all this code go!
@hatched
Owner

hatched commented Mar 10, 2015

Thanks! :shipit:
@jujugui
Member

jujugui commented Mar 10, 2015

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

jujugui added a commit that referenced this pull request Mar 10, 2015

@jujugui
Merge pull request #704 from hatched/remove-editorial 
Remove editorial.js and charmresults.js

These two files are unused, left over from the restructuring of the charm list.
f216959

@jujugui jujugui merged commit f216959 into juju:develop Mar 10, 2015
1 check failed

1 check failed

default Merged build finished.
Details

@hatched hatched deleted the hatched:remove-editorial branch Mar 23, 2015

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