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.

Sign up for GitHub

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

Jump to bottom

Update Reader.php #13

Open

steinm6 wants to merge 2 commits into joshribakoff:master from steinm6:patch-1

steinm6 commented Dec 1, 2014

Typo on line 111. skippedlines should be skippedLines


          Update Reader.php

d7c991c

Typo on line 111. skippedlines should be skippedLines

Owner

joshribakoff commented Dec 11, 2014

Thanks, looks like it wasn't fixed in all places, so the automated tests failed:

There was 1 error:

1) ReaderTest::test_Csv_Reader_Removes_Blank_Lines

Undefined property: Csv_Reader::$skippedlines

/home/travis/build/joshribakoff/php-csv-utils/source/Csv/Reader/Abstract.php:176

/home/travis/build/joshribakoff/php-csv-utils/tests/ReaderTest.php:135

https://travis-ci.org/joshribakoff/php-csv-utils/jobs/42643925


          Update Abstract.php

795907e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet