Re: Patch submission - curl_escape()

On 06/11/2013 12:08 AM, Hannes Magnusson wrote:

On Mon, Jun 10, 2013 at 1:57 PM, Adam Kazimierczak <kazymjir@gmail.com> wrote:

Hello Mailing List, This is my first contribution for the documentation project. I would ask you to review my patch and give me some tips for the future. The patch is for curl function curl_escape, located in /en/reference/curl/functions/. Using an occasion I would to ask a question - in the future should I divide patches into separate emails or rather put them into a single email?

Hah. I see you are already up and running, fantastic! I've committed your first patch, it looks great and follows all our current guidelines :D In the future, if you plan on becoming regular contributor I'd say you can just fill out the account request form once you've sent in a handful of other patches. Then you can commit them yourself rather then needing to mail them here. In the meantime, separate patches for different things (i.e. fixing a bug in the docs vs document new thing), but it can all be in the same mail. -Hannes

Thank you for the response and the commit. It's great that everything is correct. Soon I will try to submit more patches. Cheers, Adam