{"id":1558,"date":"2010-07-10T15:08:50","date_gmt":"2010-07-10T20:08:50","guid":{"rendered":"http:\/\/mikeconley.ca\/blog\/?p=1558"},"modified":"2023-12-20T16:25:16","modified_gmt":"2023-12-20T21:25:16","slug":"filing-defects-in-review-board","status":"publish","type":"post","link":"https:\/\/mikeconley.ca\/blog\/2010\/07\/10\/filing-defects-in-review-board\/","title":{"rendered":"Filing Defects in Review Board"},"content":{"rendered":"

In my last post, I talked about an extension for Review Board<\/a> that would allow users to register \u201cdefects\u201d, \u201cTODOs\u201d or \u201cproblems\u201d with code that’s up for review.<\/p>\n

After chatting with the lead RB devs for a bit, we’ve decided to scrap the extension.<\/p>\n

[audible gasp, booing, hissing]<\/p>\n

Instead, we’re just going to put it in the core of Review Board.<\/p>\n

[thundering applause]<\/p>\n

Defects<\/h3>\n

Why is this useful?\u00a0 I’ve got a few reasons for you:<\/p>\n

    \n
  1. It’ll be easier for reviewees to keep track of things left to fix, and similarly, it’ll be harder for reviewees to accidentally skip over fixing a defect that a reviewer has found<\/li>\n
  2. My statistics extension will be able to calculate useful things like defect detection rate<\/strong>, and defect density<\/strong><\/li>\n
  3. Maybe it’s just me, but checking things off as “fixed” or “completed” is really<\/em> satisfying<\/li>\n
  4. Who knows, down the line, I might code up an extension that lets you turn finding\/closing defects into a game<\/a><\/li>\n<\/ol>\n

    However, since we’re adding this to the core of Review Board<\/a>, we have to keep it simple.\u00a0 One of Review Board’s biggest strengths is in its total lack of clutter.\u00a0 No bells.\u00a0 No whistles.\u00a0 Just the things you need to get the job done.\u00a0 Let the extensions bring the bells and whistles.<\/p>\n

    So that means creating a bare-bones defect-tracking mechanism and UI, and leaving it open for extension.\u00a0 Because who knows, maybe there<\/a> are<\/a> some<\/a> people<\/a> out there who want to customize what kind <\/em>of defects they’re filing.<\/p>\n

    I’ve come up with a design that I think is pretty simple and clean.\u00a0 And it doesn’t rock the boat – if you’re not interested in filing defects, your Review Board experience stays the same.<\/p>\n

    Filing a Defect<\/h4>\n

    I propose adding a simple checkbox to the comment dialog to indicate that this comment files a defect, like so:<\/p>\n

    \"Comment<\/a>

    No bells. No whistles. Just a simple little checkbox.<\/p><\/div>\n

    While I’m in there, I’ll try to toss in some hooks so that extension developers can add more fields – for example, the classification or the priority of the defect.\u00a0 By default, however, it’s just a bare-bones little checkbox.<\/p>\n

    So far, so good.\u00a0 You’ve filed a defect.\u00a0 Maybe this is how it’ll look like in the in-line comment viewer:<\/p>\n

    \"The<\/a>

    A defect has been reported!<\/p><\/div>\n

    Two Choices<\/h4>\n

    A reviewer can file defects reports, and the reviewee<\/strong> is able to act on them.<\/p>\n

    Lets say I’m the reviewee.\u00a0 I’ve just gotten a review, and I’ve got my editor \/ IDE with my patch waiting in the background.\u00a0 I see a few defect reports have been filed.\u00a0 For the ones I completely agree with, I fix them in my editor, and then go back to Review Board and mark them as Fixed<\/strong>.<\/p>\n

    \"The<\/a>

    All fixed!<\/p><\/div>\n

    It’s also possible that I might not agree with one or more of the defect reports.\u00a0 In this case, I’ll reply to the comment to argue my case.\u00a0 I might also mark the defect report as Pass<\/strong>, which means, “I’ve seen it, but I think I’ll pass on that”.<\/p>\n

    \"The<\/a>

    I think I'll pass on that, thanks.<\/p><\/div>\n

    These comments and defect reports are also visible in the review request details page:<\/p>\n

    \"A<\/a>

    A defect has been filed.<\/p><\/div>\n

    \"The<\/a>

    All fixed up.<\/p><\/div>\n

    \"We're<\/a>

    It's all good - just pass this defect report.<\/p><\/div>\n

    Thoughts?<\/h3>\n

    What do you think?\u00a0 Am I on the right track?\u00a0 Am I missing a case?\u00a0 Does “pass” make sense?\u00a0 Will this be useful?\u00a0 I’d love to hear your thoughts.<\/p>\n","protected":false},"excerpt":{"rendered":"

    In my last post, I talked about an extension for Review Board that would allow users to register \u201cdefects\u201d, \u201cTODOs\u201d or \u201cproblems\u201d with code that’s up for review. After chatting with the lead RB devs for a bit, we’ve decided to scrap the extension. [audible gasp, booing, hissing] Instead, we’re just going to put it […]<\/p>\n","protected":false},"author":4,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"jetpack_post_was_ever_published":false,"_jetpack_newsletter_access":"","_jetpack_dont_email_post_to_subs":false,"_jetpack_newsletter_tier_id":0,"_jetpack_memberships_contains_paywalled_content":false,"_jetpack_memberships_contains_paid_content":false,"footnotes":"","jetpack_publicize_message":"","jetpack_publicize_feature_enabled":true,"jetpack_social_post_already_shared":false,"jetpack_social_options":{"image_generator_settings":{"template":"highway","default_image_id":0,"font":"","enabled":false},"version":2}},"categories":[657],"tags":[787,788,779,780,784,131,789,791,1219,504,790],"class_list":["post-1558","post","type-post","status-publish","format-standard","hentry","category-extensions-review-board-code-reviews-computer-science-technology","tag-bugs","tag-bugtracking","tag-defect-density","tag-defect-detection-rate","tag-defects","tag-extensions","tag-fixes","tag-mockups","tag-review-board","tag-reviewboard","tag-ui"],"jetpack_publicize_connections":[],"jetpack_featured_media_url":"","jetpack_shortlink":"https:\/\/wp.me\/prmTy-p8","jetpack_sharing_enabled":true,"_links":{"self":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/1558","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/users\/4"}],"replies":[{"embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/comments?post=1558"}],"version-history":[{"count":13,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/1558\/revisions"}],"predecessor-version":[{"id":1843,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/posts\/1558\/revisions\/1843"}],"wp:attachment":[{"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/media?parent=1558"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/categories?post=1558"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/mikeconley.ca\/blog\/wp-json\/wp\/v2\/tags?post=1558"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}