Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

(more) SM/EM v3 compatibility#69

Merged
weierophinney merged 8 commits intozendframework:developfrom
weierophinney:feature/68
Feb 11, 2016
Merged

(more) SM/EM v3 compatibility#69
weierophinney merged 8 commits intozendframework:developfrom
weierophinney:feature/68

Conversation

@weierophinney
Copy link
Member

This builds on #68 and #64, providing a few final touches:

  1. Allow testing against stable dependencies
    • Updated versions for:
      • zend-stdlib (allow either 2.7 or 3.0 series)
      • zend-sessionmanager (for 3.0, pin to ^3.0.3 to ensure compat trait is present)
      • zend-session ^2.5
    • Set branch-alias for develop back to 2.6-dev
    • Test both eventmanager versions!
    • Remove zend-session when testing against zend-servicemanager v3, as it's not yet forwards-compatible; skip tests for the Session adapter as well.
  2. Additionally, updates tests that introspect listeners:
    • Removed custom EventManagerIntrospectionTrait in favor of the one EventListenerIntrospectionTrait shipped with zend-eventmanager.
    • Updated all tests that used the trait to use the official one, fixing failing tests.

ezimuel and others added 8 commits February 11, 2016 13:28
- Updated versions for:
  - zend-stdlib (allow either 2.7 or 3.0 series)
  - zend-sessionmanager (for 3.0, pin to `^3.0.3` to ensure compat trait is present)
  - zend-session `^2.5`
- Set branch-alias for develop back to 2.6-dev
- Test both eventmanager versions!
- Remove zend-session when testing against zend-servicemanager v3, as it's not
  yet forwards-compatible; skip tests for the Session adapter as well.

Additionally, updates tests that introspect listeners:

- Removed custom EventManagerIntrospectionTrait in favor of the one
  EventListenerIntrospectionTrait shipped with zend-eventmanager.
- Updated all tests that used the trait to use the official one, fixing failing
  tests.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants