Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coverage off #113

Merged
merged 9 commits into from
Jun 7, 2015
Merged

Coverage off #113

merged 9 commits into from
Jun 7, 2015

Conversation

gregsymons
Copy link
Contributor

Closes #109.

Note that in addition to adding a coverageOff task, this patch merges master with the 1.1.0 release and fixes the scripted sbt tests. It also bumps the version to 1.2.0-SNAPSHOT and adds the scripted tests to the travis configuration. I can pretty easily split the other stuff out if you'd prefer a more focused patch.

@sksamuel
Copy link
Member

sksamuel commented Jun 6, 2015

This is the PR dreams are made of :)
Although I might set the version to 1.1.1 I don't think we need a big bump for a minor new task.
Also, you removed the stuff on parallel execution?

@gregsymons
Copy link
Contributor Author

I assume you mean the changes from #110? Yeah, I merged those in from master. I can change the version to 1.1.1 if you want; I tend to be more strict on the "bugfixes only" for patch releases, but it's really a personal thing.

sksamuel added a commit that referenced this pull request Jun 7, 2015
@sksamuel sksamuel merged commit e212757 into scoverage:master Jun 7, 2015
@gregsymons gregsymons deleted the coverage-off branch June 9, 2015 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Turning coverage on in the console can't be turned off
2 participants