Merge lp:~mbruzek/charms/trusty/elasticsearch/tests into lp:charms/trusty/elasticsearch

Proposed by Matt Bruzek
Status: Merged
Merged at revision: 45
Proposed branch: lp:~mbruzek/charms/trusty/elasticsearch/tests
Merge into: lp:charms/trusty/elasticsearch
Diff against target: 7 lines (+0/-1)
1 file modified
tests/tests.yaml (+0/-1)
To merge this branch: bzr merge lp:~mbruzek/charms/trusty/elasticsearch/tests
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+302853@code.launchpad.net

Description of the change

Removing "flake8" because that is not the right package name nor is it needed for the amulet tests.

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM.

I wonder how this ever passed? Seems strange that we didn't see the behavior before when it was introduced.

Thanks for the fix!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/tests.yaml'
2--- tests/tests.yaml 2016-07-14 19:55:49 +0000
3+++ tests/tests.yaml 2016-08-12 20:24:37 +0000
4@@ -1,3 +1,2 @@
5 packages:
6 - amulet
7- - flake8

Subscribers

People subscribed via source and target branches