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

Better validators #97

Merged
merged 7 commits into from Nov 7, 2017

Conversation

Projects
None yet
1 participant
@samuelcolvin
Copy link
Owner

samuelcolvin commented Nov 6, 2017

fix #18, fix #15, should also solve this discussion on #29

@codecov

This comment has been minimized.

Copy link

codecov bot commented Nov 6, 2017

Codecov Report

Merging #97 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #97   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          11     11           
  Lines         836    882   +46     
  Branches      185    199   +14     
=====================================
+ Hits          836    882   +46

@samuelcolvin samuelcolvin force-pushed the better-validators branch from 295792d to f1f670a Nov 7, 2017

@samuelcolvin samuelcolvin merged commit dfc5924 into master Nov 7, 2017

5 checks passed

codecov/project 100% (+0%) compared to 02dc2f2
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
pyup.io/safety-ci No dependencies with known security vulnerabilities.
Details

@samuelcolvin samuelcolvin deleted the better-validators branch Nov 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.