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

add unit test coverage support #22

Merged
merged 6 commits into from Nov 25, 2019

Conversation

j-richey
Copy link
Owner

Pull request that enables checking code coverage and publishing to Coveralls.

Some other project level clean up and enhancements are contained in this pull request such as clarifying how to contribute and adding stricter Clippy checks.

See: issue #21

The badge allows others to quickly know the library contains unit tests
and how much of the code is covered by said tests.

See: #21
This allows Coveralls to monitor the code coverage overtime and check
pull requests to ensure there is sufficient coverage.
Add separate document on how to contribute to the project. This is
helpful for both new and existing contributors to remind them of
the various tools and techniques used to maintain the project's code
quality.
Some of the Clippy defaults are not as strict as recommended in the
Code Complete book. The added configuration file allows the stricter
values to be configured.
The previous Travis CI configuration was resulting in long build times
from cargo-travis being installed. This update should improve the build
times from only downloading / updating outdated packages.
@j-richey j-richey merged commit 999cdef into master Nov 25, 2019
@j-richey j-richey deleted the issue-21-add-unit-test-coverage-support branch November 25, 2019 23:31
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.

None yet

1 participant