rom-rb/devtools — pull request #33 by moonglum (2013-09-13 13:39:56+00:00)
Title: Integration Specs are not a metric

Hey :smile:

I disagree a little with the way that devtools defines the different CI tasks :warning: This is extremely nitpicky :warning:

The CI namespace contains the following tasks:

* `metrics:coverage`
* `spec:integration`
* `metrics:yardstick:verify`
* `metrics:rubocop`
* `metrics:flog`
* `metrics:flay`
* `metrics:reek`

This is awesome, I totally agree with this. **But** the `ci:metrics` executes all of them – including the integration tests. Integration tests are not a metric and should therefore not be run here. This is also apparent from the description of the task "Run metrics (except mutant) and spec". The `ci` task then has to run the acceptance tests of course :wink:

I know, this is super nitpicky :blush: