Re: [PEPr] Comment on Internationalization::Arabic

Thanks Christian and David, - I agree with you that I18N_Arabic will be better name for this package, thanks for your advice, I will do it - Thanks Christian for your advice in naming sub classes using PEAR standard, I think it should be I18N_Arabic_$class - Dear David, source code already hosted on sourceforge.net, sorry because I did not mention that, you can find it here: http://sourceforge.net/projects/ar-php/ Well, I wish to shift it into github.com, but I don't know an easy way to import my CVS repository and keep developing history - Dear David, thanks for your advices to use PHP_CodeSniffer, I will do scan all may files using that tool fix all reported issues, I suppose using following command will be enough to satisfy PEAR standards: phpcs --standard=Zend -n $filename So, I will do my homework then back to you :o) Best regards, Khaled On Wed, Nov 4, 2009 at 2:22 PM, David Coallier <david@echolibre.com> wrote: > > - If I were you I would read: > http://pear.php.net/manual/en/standards.php > and to make sure your code complies to the coding standards you can use > PHP_CodeSniffer which will help you identify parts of the code that do not > comply to PEAR coding standards > > - I do think I18N_Arabic could be a better name as well > > - What you could do is put your code on github.com and from there we will > be able to fork and give you tips (Seeing it online instead of downloading > will greatly improve the amount of comments you'll receive) > > -- > http://pear.php.net/pepr/pepr-proposal-show.php?id=617 > -- http://www.ar-php.org PHP and Arabic language solutions by Khaled Al-Shamaa