Skip to content

rchk protect#2836

Merged
mattdowle merged 6 commits intomasterfrom
rchk
May 5, 2018
Merged

rchk protect#2836
mattdowle merged 6 commits intomasterfrom
rchk

Conversation

@mattdowle
Copy link
Copy Markdown
Member

@mattdowle mattdowle commented May 4, 2018

WIP. Will go file by file in this PR.
Closes #2835
Did not help with #2767.

@mattdowle mattdowle added this to the 1.11.2 milestone May 4, 2018
@codecov-io
Copy link
Copy Markdown

codecov-io commented May 4, 2018

Codecov Report

Merging #2836 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2836      +/-   ##
==========================================
+ Coverage   93.45%   93.46%   +0.01%     
==========================================
  Files          61       61              
  Lines       12358    12348      -10     
==========================================
- Hits        11549    11541       -8     
+ Misses        809      807       -2
Impacted Files Coverage Δ
src/rbindlist.c 88.97% <100%> (+0.02%) ⬆️
src/freadR.c 89.4% <100%> (-0.25%) ⬇️
src/dogroups.c 93.47% <100%> (ø) ⬆️
src/fmelt.c 84.59% <100%> (-0.07%) ⬇️
src/assign.c 94.3% <100%> (-0.02%) ⬇️
src/fsort.c 73.93% <0%> (+1.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf0d4e7...efe20f1. Read the comment docs.

@mattdowle mattdowle merged commit 8b61cbf into master May 5, 2018
@mattdowle mattdowle deleted the rchk branch May 5, 2018 01:17
This was referenced May 5, 2018
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.

2 participants