lazr.restful

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

  1. add-destructor-operation-to-all-decl
  2. Merge into trunk
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
1=== modified file 'src/lazr/restful/declarations.py'
2--- src/lazr/restful/declarations.py 2009-06-08 20:03:55 +0000
3+++ src/lazr/restful/declarations.py 2009-08-14 16:11:34 +0000
4@@ -17,6 +17,7 @@
5 'exported',
6 'export_as_webservice_collection',
7 'export_as_webservice_entry',
8+ 'export_destructor_operation',
9 'export_factory_operation',
10 'export_operation_as',
11 'export_read_operation',

Subscribers

People subscribed via source and target branches