Merge lp:~adeuring/lazr.restful/add-destructor-operation-to-all-decl into lp:lazr.restful

Proposed by Abel Deuring
Status: Merged
Merged at revision: not available
Proposed branch: lp:~adeuring/lazr.restful/add-destructor-operation-to-all-decl
Merge into: lp:lazr.restful
Diff against target: None lines
To merge this branch: bzr merge lp:~adeuring/lazr.restful/add-destructor-operation-to-all-decl
Reviewer Review Type Date Requested Status
Guilherme Salgado (community) Approve
Review via email: mp+10168@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Abel Deuring (adeuring) wrote :

When I tried to use @export_destructor_operation() in a Launchpad branch, I noticed the warning

  There were 1 imports of names not appearing in the __all__.
  You should not import export_destructor_operation from lazr.restful.declarations:
      canonical.launchpad.interfaces.hwdb

This is the trivial fix for the wrning.

Revision history for this message
Guilherme Salgado (salgado) wrote :

 review approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/lazr/restful/declarations.py'
--- src/lazr/restful/declarations.py 2009-06-08 20:03:55 +0000
+++ src/lazr/restful/declarations.py 2009-08-14 16:11:34 +0000
@@ -17,6 +17,7 @@
17 'exported',17 'exported',
18 'export_as_webservice_collection',18 'export_as_webservice_collection',
19 'export_as_webservice_entry',19 'export_as_webservice_entry',
20 'export_destructor_operation',
20 'export_factory_operation',21 'export_factory_operation',
21 'export_operation_as',22 'export_operation_as',
22 'export_read_operation',23 'export_read_operation',

Subscribers

People subscribed via source and target branches