← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #16060

Re: [Merge] lp:~phill-ridout/openlp/issue-507 into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Needs Fixing

9 +from lxml import objectify, etree

No need for the etree import.

27 +

Double new line.

Please also add a docstring to your method.
-- 
https://code.launchpad.net/~phill-ridout/openlp/issue-507/+merge/110682
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next