From: Brian M. <br...@gr...> - 2008-07-01 03:06:51
|
Craig, > I regret to say that no. The patch is NOT small. or at > least in my opinion. > And yes. I would like a review of this report. That would > be excellent. > And I was hoping for a merge into the trunk too (after > review). If you would like your report to become part of the official distribution, here are some things to keep in mind: The Descendant Tree report and the Ancestor Tree report are meant to be consistent with each other. So any features that are added to the Descendant Tree should be added to the Ancestor Tree if possible. Also, have a look at our coding standards: http://www.gramps-project.org/wiki/index.php?title=Programming_Guidelines The most important part (in my opinion is PEP8). Thanks again for the contributions. We really appreciate it! ~Brian |