[petsc-dev] Patch review
Barry Smith
bsmith at mcs.anl.gov
Fri Aug 24 17:51:49 CDT 2012
On Aug 24, 2012, at 5:42 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:
>
> I'm a little concerned that the current review process is all ex post facto. It makes more work for reviewers and the history ends up looking confusing. I think we should work out a review process that can happen before things are pushed to petsc-dev, so that the authors of the patches can clean them up in response to comments. When done right, this ends up being less work for both the reviewers and the authors.
Propose a system.
Barry
Just make sure it doesn't involve Sean holding the master password for the petsc repository and then getting hacked again.
More information about the petsc-dev
mailing list