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@...> 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.orgPHP and Arabic language solutions by Khaled Al-Shamaa
