際際滷

際際滷Share a Scribd company logo
3     better
 steps to

code review
KATHRYN ROTONDO
lets meet &
review code!
ugh
3 steps to better code review
we three easy stepsit
 in
    can fix
1. STOP meeting
3 steps to better code review
2. USE tools
ide plugins
flex formatter
       or
    anyedit
web tools
github pull requests
         or
google code reviews
3. EVERY commit
package {

    import flash.display.sprite;

    public class HelloWorld extends Sprite() {

        public function helloworld () {
          trace (Hello World);
        }
    }
}
package {

    import flash.display.sprite;

    public class HelloWorld extends Sprite() {

        public function helloworld () {
          trace (Hello World);
        }
    }
}
package {

    import flash.display.sprite;

    public class HelloWorld extends Sprite() {

        public function helloworld () {   sprite behavior
                                          coming soon?
          trace (Hello World);
        }                                 if so, looks good!
                                                     Reviewer
    }
}
package {

    import flash.display.sprite;

    public class HelloWorld extends Sprite() {

        public function helloworld () {   sprite behavior
                                          coming soon?
          trace (Hello World);
        }                                 if so, looks good!
                                                     Reviewer
    }
}                                         yes, coming soon.
                                                    Developer
lather, rinse,
   repeat
push
3     better
 steps to

code review
KATHRYN ROTONDO

More Related Content

3 steps to better code review

  • 1. 3 better steps to code review KATHRYN ROTONDO
  • 3. ugh
  • 5. we three easy stepsit in can fix
  • 10. flex formatter or anyedit
  • 12. github pull requests or google code reviews
  • 14. package { import flash.display.sprite; public class HelloWorld extends Sprite() { public function helloworld () { trace (Hello World); } } }
  • 15. package { import flash.display.sprite; public class HelloWorld extends Sprite() { public function helloworld () { trace (Hello World); } } }
  • 16. package { import flash.display.sprite; public class HelloWorld extends Sprite() { public function helloworld () { sprite behavior coming soon? trace (Hello World); } if so, looks good! Reviewer } }
  • 17. package { import flash.display.sprite; public class HelloWorld extends Sprite() { public function helloworld () { sprite behavior coming soon? trace (Hello World); } if so, looks good! Reviewer } } yes, coming soon. Developer
  • 18. lather, rinse, repeat
  • 19. push
  • 20. 3 better steps to code review KATHRYN ROTONDO