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

Travis-CI integration #20

Closed
wants to merge 16 commits into from
Closed

Travis-CI integration #20

wants to merge 16 commits into from

Conversation

Pr0methean
Copy link

@Pr0methean Pr0methean commented Aug 9, 2017

Fixes #18
This is one of the cherrypicks of #16. It updates Cobertura, breaks out the dieharder dependency to /usr/bin so that it's more cross-platform (still won't work on MinGW or naked Windows), switches DiehardInputGenerator to use a named pipe to reduce disk space requirements, and splits the Dieharder suite into two so that it'll finish within a free Travis-CI.org instance's timeout. It also decreases the number of bytes demanded from random.org.

Branch coverage is currently below spec at 93.4%, even though all the code I've touched for this test has full branch coverage! Possibly some exception-handling code in RandomDotOrgSeedGenerator is no longer being exercised now that we're not overusing the service?

@Pr0methean
Copy link
Author

Withdrawing this PR, because the improvements I've made are probably too thorough to merge into a project without much heavier active maintenance. Instead, I've forked the relevant parts into https://github.com/Pr0methean/betterrandom.

@Pr0methean Pr0methean closed this Sep 9, 2017
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.

1 participant