Skip to content

Fix countfields() on the last line of file - #2544

Merged
mattdowle merged 2 commits into
masterfrom
issue2543
Jan 3, 2018
Merged

mattdowle merged 2 commits into
masterfrom
issue2543

Conversation

@st-pasha

@st-pasha st-pasha commented Jan 2, 2018

Copy link
Copy Markdown
Contributor

Closes #2543

@st-pasha st-pasha added the fread label Jan 2, 2018
@st-pasha st-pasha added this to the v1.10.6 milestone Jan 2, 2018
@st-pasha st-pasha self-assigned this Jan 2, 2018
@st-pasha
st-pasha requested a review from mattdowle January 2, 2018 22:41
@codecov-io

codecov-io commented Jan 3, 2018 •

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2544   +/-   ##
=======================================
  Coverage   91.45%   91.45%           
=======================================
  Files          63       63           
  Lines       12071    12071           
=======================================
  Hits        11039    11039           
  Misses       1032     1032
Impacted Files Coverage Δ
src/fread.c 95.97% <100%> (ø) ⬆️

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 2a9ce61...debd823. Read the comment docs.

@mattdowle
mattdowle merged commit f1d10c5 into master Jan 3, 2018
@mattdowle
mattdowle deleted the issue2543 branch January 3, 2018 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants