openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #18011
Re: [Merge] lp:~marmyshev/openlp/autoplay into lp:openlp
Review: Needs Fixing
99, 114, 129.
The method names should be lowercase_with_underscores rather than camelCase (unless its QT derived)
http://wiki.openlp.org/Development:Coding_Standards#Naming_Conventions_2
I know not all methods conform to this standard.
Can you make the doc strings more meaningful please.
--
https://code.launchpad.net/~marmyshev/openlp/autoplay/+merge/133739
Your team OpenLP Core is subscribed to branch lp:openlp.
References