zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #14531
Re: [Merge] lp:~zorba-coders/zorba/1025564 into lp:zorba
Review: Needs Fixing
Works great. Some minor issues:
- the --help documentation should mention that the argument is deprecated
- also, the --help documentation should mention the syntax to pass the file
- please respect coding guidelines. For example,
if(qBeforeF()){ => if (qBeforeF())
{
- remove commented out code
--
https://code.launchpad.net/~zorba-coders/zorba/1025564/+merge/125256
Your team Zorba Coders is subscribed to branch lp:zorba.
References