On Tue, May 4, 2010 at 1:58 AM, Robert Bradshaw <[email protected]> wrote: > On May 3, 2010, at 8:40 AM, Craig Citro wrote: > >>> Just a suggestion, take it or leave it: >>> >>> For my GSoC work I found google's codereview to be a very helpful >>> tool. >>> >>> http://code.google.com/p/rietveld/wiki/CodeReviewHelp >>> >>> It allows you to submit patches that are presented in an app-engine >>> GUI. Reviewers can make comments & suggestions that are local to >>> each >>> change in the patch. >>> >>> Perhaps it would be easier to use for interactive patch review than >>> trac. (It works seamlessly with mercurial, too). >>> >> >> +10 -- we should *totally* start using rietveld. (Plus, in a month or >> two, Robert and I will be totally in love with it.) > > > I already am :). It may be good to put up a repo on hg.cython.org (so > it's easy for people to sync up with what you have to try it out and > play with it). Also, did you set up a blog yet? > > - Robert >
Hi, My blog that dedicated to the Cython GSoC work has been set up at here: http://haoyugsoc.wordpress.com/ I'll post weekly update onto this blog. It would be cool for me to have a repo on hg.cython.org, though Craig and me were thinking that since the work involves many small patches so it would be a lot of lightweight branches inside the repo, which would be not easy for others to try it out. However, a repo would be good to host and backup my work and help me sync among my computers. So, then how to get the repo setup? Meanwhile I have play around rietveld. An issue for Cython ticket #422 is created here: (with a revised patch) http://codereview.appspot.com/1228043/show Seems there is no tight integration of trac and rietveld. For now I will have the issue description in rietveld be a url point to Cython trac issue. I'll see if I can have some simple scripts to automate the workflow a bit more. :) Thanks! -- Haoyu BAI School of Computing, National University of Singapore. _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
