-
Notifications
You must be signed in to change notification settings - Fork 61
Remove stream wrapper functionality #34
base: develop
Are you sure you want to change the base?
Conversation
Tests are failing, but it looks like compatibility issues with other ServiceManager v3. |
As I said on #23, |
@sasezaki the reason this is labeled as BC break is removal of API (even if protected). The correct approach could be deprecation + removal of these method bodies, while the API survives till the next major release. |
@Ocramius thanks describe reason |
@sasezaki it's an API change, which means that any subclasses may be broken by the change (that's why we use |
This repository has been closed and moved to laminas/laminas-view; a new issue has been opened at laminas/laminas-view#28. |
This repository has been moved to laminas/laminas-view. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
Fixes #23.