← Back to team overview

openlp-core team mailing list archive

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

Re: [Merge] lp:~googol/openlp/short-lines into lp:openlp

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

Looks fine, only one minor issue I see so far:

353	+"""
354	+The :mod:`openlp.core.utils` module provides the utility libraries for OpenLP.
355	+"""

Your docstring is not correct :-)
-- 
https://code.launchpad.net/~googol/openlp/short-lines/+merge/150784
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next