Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mrxvt - Multi-tabbed terminal emulator. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223422 mtasaka@xxxxxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| | Flag|fedora-review? |fedora-review+ ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-06-18 10:02 EST ------- One comment: * Umm.. actually the "review style" is difficult issue. Many reviewers use some check list styles of their own, and it may have the benefit that it may show what the reviewers actually checked. Some other reviewers complain if there is no check list written on the review. However, there are also opinitions that the long verbose list is the same as just a copy/paste and makes it difficult to watch. So while the check list can exist, especially by the time you get used to reviewing, you should put the summary *at the end* to show briefly what the submitter should fix. But the most good way is to try anyway!! Again, reviewing style is somewhat difficult issue and actually there was a discussion about reviewing style (Some said that there should be check lists, some said that it is not needed) Well, ------------------------------------------------------------ This package (mrxvt) is APPROVED by me ------------------------------------------------------------ Please follow the procedure written on http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account" When you requested someone to sponsor you (in the procedure above), please make a note on this bug that you did so. If you want to push this package also on F-7, you also have to check: http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT after the URL above. !! Well, recenctly Fedora package system changed a lot !! If you have some questions, please let me know. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review