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

Refactor/164 removeoldcollection #170

Closed
wants to merge 51 commits into from

Conversation

Projects
None yet
1 participant
@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 10, 2016

Merge pull request #163 from nozavroni/analysis-8n0wP5
Apply fixes from StyleCI - First time StyleCI auto-fix PR
Merge pull request #162 from nozavroni/refactor/141
Collection refactor - AbstractCollection, MultiCollection, TabularCollection, etc.
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

@nozavroni nozavroni added the refactor label Dec 12, 2016

@nozavroni nozavroni added this to the v0.2.2 Release milestone Dec 12, 2016

@nozavroni nozavroni self-assigned this Dec 12, 2016

@nozavroni nozavroni closed this Dec 12, 2016

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.