Hi John,

For most open-source projects, code review is done by the committers or core developers.  That's true as well for Yorba's projects.

If you're interested in contributing, I would recommend reading the following pages on our wiki:

http://redmine.yorba.org/projects/yorba/wiki/ExternalContributors
http://redmine.yorba.org/projects/yorba/wiki/CodingConventions
http://redmine.yorba.org/projects/yorba/wiki/PatchSubmissionCodeReviews

If you submit a patch, you'll go through our review process, but on the other side, as someone whose code is being reviewed.  Still, it might give you an idea of the process and how we, at least, interpret it.

-- Jim


On Sat, Mar 16, 2013 at 3:05 PM, John Kim <[email protected]> wrote:
Hello,

If there is one thing about development I want to understand, it's
reviewing code or patches. I see that some issues are ready for review? Is anyone allowed to do the review, developer or contributor? If so, how
can he apply the patch and test it?

I'm interested in reviewing some issues, though I'm not sure of who gets to
do it or what must be done.

John
_______________________________________________
Geary mailing list
[email protected]
http://lists.yorba.org/cgi-bin/mailman/listinfo/geary

_______________________________________________
Geary mailing list
[email protected]
http://lists.yorba.org/cgi-bin/mailman/listinfo/geary

Reply via email to