Clean-up shinkentest and test-cases: Use a mix-in class instead of ugly code. #608

Closed
wants to merge 6 commits into
from

Conversation

Projects
None yet
5 participants
Contributor

htgoebel commented Sep 15, 2012

No description provided.

Contributor

xkilian commented Sep 18, 2012

@naparuba What do you think about this patch? Is it worth testing and adding to 1.2.1?

Owner

naparuba commented Sep 18, 2012

If tests are good, commit :)

Jean

On Tue, Sep 18, 2012 at 3:45 AM, xkilian notifications@github.com wrote:

@naparuba https://github.com/naparuba What do you think about this
patch? Is it worth testing and adding to 1.2.1?


Reply to this email directly or view it on GitHubhttps://github.com/naparuba/shinken/pull/608#issuecomment-8637097.

Contributor

htgoebel commented Sep 18, 2012

No need to put this into 1.2.1 as it only effects the test-cases.

Collaborator

Frescha commented Jan 26, 2013

any updates?

Contributor

Seb-Solon commented Jun 11, 2013

What about merging this?

@Seb-Solon Seb-Solon modified the milestones: 2.2 (Maleable Muskrat) **doc, cli & arbiters**, Not prioritized Apr 21, 2014

@Seb-Solon Seb-Solon self-assigned this Apr 21, 2014

@Seb-Solon Seb-Solon referenced this pull request Apr 21, 2014

Closed

Shinken 2.2 Roadmap #1175

6 of 6 tasks complete

Seb-Solon added a commit that referenced this pull request May 3, 2014

@Seb-Solon Seb-Solon closed this in 82ca75e May 3, 2014

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