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

WIP: Improve test coverage #6

Merged
merged 3 commits into from Jun 19, 2016

Conversation

@krlmlr
Copy link
Member

krlmlr commented Mar 8, 2016

More to follow, aiming for full coverage.

@krlmlr krlmlr force-pushed the krlmlr:feature/travis branch from 881ebce to c772807 Jun 19, 2016
Kirill Müller added 2 commits Mar 8, 2016
Kirill Müller
@krlmlr krlmlr force-pushed the krlmlr:feature/travis branch from 55adf66 to bc7b366 Jun 19, 2016
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jun 19, 2016

Current coverage is 77.63%

Merging #6 into master will increase coverage by 11.24%

@@             master         #6   diff @@
==========================================
  Files             8          8          
  Lines           241        228    -13   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            160        177    +17   
+ Misses           81         51    -30   
  Partials          0          0          

Powered by Codecov. Last updated by d235839...2144870

@krlmlr krlmlr merged commit 4551e6d into tidyverse:master Jun 19, 2016
2 checks passed
2 checks passed
codecov/project 77.63% (+11.24%) compared to d235839
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@krlmlr krlmlr deleted the krlmlr:feature/travis branch Jun 19, 2016
krlmlr pushed a commit that referenced this pull request Jun 19, 2016
Kirill Müller
- `distinct()` gains `.keep_all` argument (#30, #31).
- Slightly improve test coverage (#6).
- Install `devtools` from GitHub on Travis (#32).
- Joins return `data.table`. Right and full join are now implemented (#16, #19).
- Remove warnings from tests (#4).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.