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

Develop #175

Merged
merged 64 commits into from Dec 12, 2016

Conversation

Projects
None yet
2 participants
@nozavroni
Copy link
Owner

commented Dec 12, 2016

No description provided.

nozavroni added some commits Nov 6, 2016

Changed name of IO\Resource class to IO\StreamResource because of PHP…
…7's "soft" restriction on the word "resource". resolves #143
AbstractCollection::factory() will now do its best to return the most…
… appropriate collection type for every possible case (for whatever you pass to it).

nozavroni added some commits Dec 11, 2016

Scrutinizer fixes:
 * Removed a bunch of dead code in TabularCollection
 * Added more consistent type checking in functions.php
 * Added exception tests for various constructors
Changed docblock type of Reader::$current to Row|null and set it to n…
…ull where before it was setting to false
Merge pull request #172 from nozavroni/analysis-XVj37B
Apply auto-fixes from StyleCI
@coveralls

This comment has been minimized.

Copy link

commented Dec 12, 2016

Coverage Status

Coverage increased (+3.1%) to 96.505% when pulling e47c82a on develop into 677afc8 on master.

@nozavroni nozavroni merged commit 01c55b3 into master Dec 12, 2016

6 checks passed

Scrutinizer 2 new issues, 161 updated code elements
Details
continuous-integration/styleci/pr The StyleCI analysis has passed
Details
continuous-integration/styleci/push The StyleCI analysis has passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+3.1%) to 96.505%
Details
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.