<div dir="ltr">Sorry to nag about this again, but can someone have a look at the open changes on lodgeit?<div><br></div><div>The important ones listed in my previous mail were merged (thanks), but there are several minor fixes pending for some time now. Some of them already have Code-Review+2 from one reviewer.<br><div><br></div><div><a href="https://review.openstack.org/#/q/project:openstack-infra/lodgeit+status:open">https://review.openstack.org/#/q/project:openstack-infra/lodgeit+status:open</a></div><div><br></div><div>Thanks.</div><div><br></div><div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 15, 2015 at 8:55 PM David Pursehouse <<a href="mailto:david.pursehouse@gmail.com">david.pursehouse@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>Does anyone have time to take a look at these open changes on openstack-infra/lodgeit?</div><div><br></div><div>- "Fix up the tests and add tox configuration to run them" [1]</div><div>When this one is merged it will be possible to enable the pep27 gate job that will run the test suite.</div><div><br></div><div>- "Import make_app from lodgeit.application instead of lodgeit" [2]</div><div>This fixes a bug (introduced by me during pep8 cleanup) that prevents the app from being run through wsgi.</div><div><br></div><div>- "Add pytz and MarkupSafe into requirements.txt" [3]</div><div>This is needed to be able to deploy the app on a clean system.  It seems they are not installed as dependencies of the existing requirements.</div><div><br></div><div>Thanks.</div><div><br></div><div><br></div><div>[1] <a href="https://review.openstack.org/#/c/234637/" target="_blank">https://review.openstack.org/#/c/234637/</a></div><div>[2] <a href="https://review.openstack.org/#/c/250643/" target="_blank">https://review.openstack.org/#/c/250643/</a></div><div>[3] <a href="https://review.openstack.org/#/c/250690/" target="_blank">https://review.openstack.org/#/c/250690/</a></div><div><br></div></div></blockquote></div></div></div></div>