Merge lp:~stevenk/storm/storm-export-propertycolumn into lp:storm

Proposed by Steve Kowalik
Status: Needs review
Proposed branch: lp:~stevenk/storm/storm-export-propertycolumn
Merge into: lp:storm
Diff against target: 12 lines (+1/-1)
1 file modified
storm/properties.py (+1/-1)
To merge this branch: bzr merge lp:~stevenk/storm/storm-export-propertycolumn
Reviewer Review Type Date Requested Status
Storm Developers Pending
Storm Developers Pending
Review via email: mp+71144@code.launchpad.net

Description of the change

Due to the below error, I am exporting PropertyColumn.

** 1 import policy violations **

There were 1 imports of names not appearing in the __all__.
You should not import PropertyColumn from storm.properties:
    canonical.launchpad.webapp.batching

To post a comment you must log in.
Revision history for this message
Jamu Kakar (jkakar) wrote :

I don't have a problem with this change, but as far as I know,
PropertyColumn is something internal to Storm and shouldn't be needed
externally. I wonder if this is something worth changing or if we
really want to keep PropertyColumn private?

Unmerged revisions

397. By Steve Kowalik

Export PropertyColumn in __all__.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'storm/properties.py'
--- storm/properties.py 2009-02-04 06:33:24 +0000
+++ storm/properties.py 2011-08-11 04:12:39 +0000
@@ -36,7 +36,7 @@
36__all__ = ["Property", "SimpleProperty",36__all__ = ["Property", "SimpleProperty",
37 "Bool", "Int", "Float", "Decimal", "RawStr", "Unicode",37 "Bool", "Int", "Float", "Decimal", "RawStr", "Unicode",
38 "DateTime", "Date", "Time", "TimeDelta", "UUID", "Enum",38 "DateTime", "Date", "Time", "TimeDelta", "UUID", "Enum",
39 "Pickle", "List", "PropertyRegistry"]39 "Pickle", "List", "PropertyColumn", "PropertyRegistry"]
4040
4141
42class Property(object):42class Property(object):

Subscribers

People subscribed via source and target branches

to status/vote changes: