Merge lp://staging/~cjwatson/storm/zope-interface-class-decorators into lp://staging/storm

Proposed by Colin Watson
Status: Merged
Merged at revision: 501
Proposed branch: lp://staging/~cjwatson/storm/zope-interface-class-decorators
Merge into: lp://staging/storm
Diff against target: 74 lines (+6/-7)
3 files modified
setup.py (+1/-0)
storm/zope/zstorm.py (+3/-5)
tests/zope/adapters.py (+2/-2)
To merge this branch: bzr merge lp://staging/~cjwatson/storm/zope-interface-class-decorators
Reviewer Review Type Date Requested Status
Facundo Batista (community) Approve
Storm Developers Pending
Review via email: mp+368445@code.staging.launchpad.net

Commit message

Use zope.interface class decorators rather than class advice.

Description of the change

Class advice doesn't work in Python 3.

The required version of zope.interface has been packaged as far back as Ubuntu 13.04 (raring), which I think is acceptable at this point.

To post a comment you must log in.
Revision history for this message
Facundo Batista (facundo) wrote :

Great!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: