Re: [PEPr] Comment on Web Services::Services_Twilio

Hi Matthew, Thanks for your comments! We're working on shaping it up to PEAR standards. I hope nobody minds too much that it's pushed out in pre-alpha form: we thought it'd be best to get some feedback from the community as early as possible. We thought about using HTTP_Request2, but that would create an extra dependency. The current request class is really a thin cURL wrapper, so only the curl extension is required. If that happens to fly directly in the face of PEAR convention, please let me know and I'll have a talk with the team here about additional deps. Thanks again! On Mon, Mar 7, 2011 at 6:14 PM, Matthew Fonda <mfonda@php.net> wrote: > > Looks like a useful package! > A couple comments after quickly looking through the code: > > 1. Make sure to follow PEAR coding standards > 2. Add docblock comments > 3. Instead of including your own HTTP request class with the package, > consider using PEAR HTTP_Request2. > > -- > http://pear.php.net/pepr/pepr-proposal-show.php?id=656 >