Checkbox

Merge lp:~kissiel/checkbox/jinja-exporter into lp:checkbox

  1. jinja-exporter
  2. Merge into trunk
Proposed by Maciej Kisielewski
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 3636
Merged at revision: 3636
Proposed branch: lp:~kissiel/checkbox/jinja-exporter
Merge into: lp:checkbox
Diff against target: 15 lines (+4/-1)
1 file modified
plainbox/plainbox/impl/exporter/jinja2.py (+4/-1)
To merge this branch: bzr merge lp:~kissiel/checkbox/jinja-exporter
Reviewer Review Type Date Requested Status
Sylvain Pineau Approve
Review via email: mp+254776@code.launchpad.net

Description of the change

This MR adds missing option list to dict returned by get_session_data_subset in Jinja2 exporter.

e122349 plainbox:exporter: make get_session_data_subset return options in jinja2 exp.

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plainbox/plainbox/impl/exporter/jinja2.py'
2--- plainbox/plainbox/impl/exporter/jinja2.py 2015-03-31 13:56:57 +0000
3+++ plainbox/plainbox/impl/exporter/jinja2.py 2015-03-31 14:58:09 +0000
4@@ -77,7 +77,10 @@
5
6 def get_session_data_subset(self, session_manager):
7 """Compute a subset of session data."""
8- return {'manager': session_manager}
9+ return {
10+ 'manager': session_manager,
11+ 'options': self.option_list,
12+ }
13
14 def supported_option_list(cls):
15 """ Return list of supported exporter options."""

Subscribers

People subscribed via source and target branches