Skip to content
This repository has been archived by the owner on Apr 23, 2020. It is now read-only.

Ignore features on the fly #111

Closed
wants to merge 7 commits into from

Conversation

debona
Copy link
Contributor

@debona debona commented Mar 4, 2014

Allow to ignore features of a suite test on the fly by providing { "without" : [1, 4] } in config.

@MattiSG
Copy link
Owner

MattiSG commented Mar 4, 2014

Thank you very much ;)

I will try to fix #112 before merging this. If it's too long, I'll merge as it is.

@MattiSG
Copy link
Owner

MattiSG commented Mar 5, 2014

#112 has been fixed (in branch v0.6.2). I will uncomment the tests you provided, unless you prefer to do it yourself.

@MattiSG
Copy link
Owner

MattiSG commented Mar 5, 2014

Even though I really like without as a CLI switch, I'm wondering if ignore would not be a more proper switch in a config file. What do you think?

@MattiSG
Copy link
Owner

MattiSG commented Mar 5, 2014

Added some commits in branch ignore-features, simply about uniforming error handling and improving messages.

Waiting for your feedback on the switch name before merging and publishing v0.6.2 :)

@debona
Copy link
Contributor Author

debona commented Mar 5, 2014

--ignore switch sounds a little bit more GNUish and it would be clearer in a config file. Even if I like --without too. ^^

@debona
Copy link
Contributor Author

debona commented Mar 5, 2014

You just have to git revert 8beb67d to enable the integration tests.
I let you handle merges to deliver that in v0.6.2.

@MattiSG
Copy link
Owner

MattiSG commented Mar 6, 2014

Merged in 51faa9c, published since 0.6.2-alpha.2. Thanks! :)

@MattiSG MattiSG closed this Mar 6, 2014
@MattiSG
Copy link
Owner

MattiSG commented Jul 21, 2014

Released and acknowledged, thanks again! :)

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.

2 participants