-
Notifications
You must be signed in to change notification settings - Fork 27
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 Dependabot and Coveralls #70
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
Do not merge until #69 is approved and merged |
ab4ade0
to
75dce76
Compare
Rebased post merging of #69 |
@highb I think because of overlapping roles this is stuck waiting on your approval too. (we may also need to adjust the way the codeowners stuff is defined) |
Actually, I just figured out the problem... fixing codeowners now |
This adds Dependabot to keep things updated and Coveralls so we can see how code coverage changes over time. Coveralls / SimpleCov are configured so that both the SimpleCov html report and the Coveralls report are generated. This facilitates easily seeing coverage locally in addtion to via CI.
Fixed duplicate testing by removing unneeded webhook integration and by updating travis to only build PR |
This PR was previously approved but rebasing busted the reviews. Merging anyhow. |
This adds Dependabot to keep things updated and Coveralls so we can see how code coverage changes over time. Coveralls / SimpleCov are configured so that both the SimpleCov html report and the Coveralls report are generated. This facilitates easily seeing coverage locally in addtion to via CI.
This PR builds on the code in #69