Skip to content

Default reporters lcov, json with a way to specify it from runner#115

Merged
kof merged 2 commits into
qunitjs:masterfrom
hakubo:Accept-coverage.reporters
Jan 8, 2015
Merged

Default reporters lcov, json with a way to specify it from runner#115
kof merged 2 commits into
qunitjs:masterfrom
hakubo:Accept-coverage.reporters

Conversation

@hakubo
Copy link
Copy Markdown
Contributor

@hakubo hakubo commented Dec 30, 2014

I need to generate cobertura report for jenkins
with this users can pass reporters to istanbul

coverage: {
  dir: 'dir',
  reporters: ['cobertura', 'lcov']
}

@fyockm
Copy link
Copy Markdown
Contributor

fyockm commented Dec 30, 2014

LGTM

@kof
Copy link
Copy Markdown
Contributor

kof commented Jan 3, 2015

thanks for PR, could you also please add some documentation to readme?

@hakubo
Copy link
Copy Markdown
Contributor Author

hakubo commented Jan 8, 2015

Is the change to docs enough? Should I clarify it better?

@kof
Copy link
Copy Markdown
Contributor

kof commented Jan 8, 2015

no its fine, thanks

kof added a commit that referenced this pull request Jan 8, 2015
@kof
Merge pull request #115 from hakubo/Accept-coverage.reporters
020b464 
Default reporters lcov, json with a way to specify it from runner
@kof kof merged commit 020b464 into qunitjs:master Jan 8, 2015
@hakubo hakubo deleted the Accept-coverage.reporters branch January 9, 2015 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hakubo @fyockm @kof