ENCODE code review process: Difference between revisions

From genomewiki
Jump to navigationJump to search
Line 14: Line 14:


== cv.ra file ==
== cv.ra file ==
* Melissa will review all cv.ra commits from the previous week, on the alpha version.
* CV Maven (Melissa) will review all cv.ra commits from the previous week, on the alpha version.
* In separate processing, Melissa will be periodically copying changes from alpha to beta.
* In separate processing, cv maven will be periodically copying changes from alpha to beta, when things look stable. She will use a diff on alpha and beta to make an extensive commit message about the types of changes being committed.  This will allow QA to have a summary of the changes they are looking at
* Katrina will review Melissa's cv.ra commits from the previous week, on beta.   
* Katrina will review Melissa's cv.ra commits from the previous week, on beta.   
* When Katrina finds issues, Melissa will address the issues both on alpha and beta.
* When Katrina finds issues, Melissa will address the issues both on alpha *and beta*.
* Once Katrina approves of the changes in beta/cv.ra, she will copy it to public/cv.ra and make public, thereby releasing the changes to the public. The next time trackDb & friends is pushed, the changes to cv.ra will be released. No review of changes to public/cv.ra is necessary since these exact changes were reviewed by Katrina in beta/cv.ra and Melissa in alpha/cv.ra.
* Once Katrina approves of the changes in beta/cv.ra, she will copy it to public/cv.ra and make public, thereby releasing the changes to the public. The next time trackDb & friends is pushed, the changes to cv.ra will be released. No review of changes to public/cv.ra is necessary since these exact changes were reviewed by Katrina in beta/cv.ra and Melissa in alpha/cv.ra.

Revision as of 22:10, 26 July 2011

ENCODE Wranglers

NOTE that sometimes ENCODE Developers perform wrangling duties -- when they do, the first two rules apply to them as well.

  • When reviewing a wrangler's check-ins, you must thoroughly check: .daf, scripts (perl, python, etc.)
  • When reviewing a wrangler's check-ins, for the sake of consistency and spreading general knowledge about the project, you should lightly check html, configuration and notes files such as: .ra, notes, .txt, .html
  • Until further notice, Kate will review wrangler's code (.c) check-ins.

ENCODE Developers

  • When reviewing an ENCODE developer's check-ins, don't gloss over ENCODE pipeline edits.
  • When reviewing an ENCODE developer's check-ins, you do not need to review edits to the cv.ra file.

ENCODE QAers

NOTE that as ENOCDE QAers do the actual QA of the tracks they will, by default, check all other changes that have been made by any ENCODEr to .ra and .html files.

  • ENCODE QAers will review all check-ins (.html, .ra, etc) *except* edits to cv.ra.

cv.ra file

  • CV Maven (Melissa) will review all cv.ra commits from the previous week, on the alpha version.
  • In separate processing, cv maven will be periodically copying changes from alpha to beta, when things look stable. She will use a diff on alpha and beta to make an extensive commit message about the types of changes being committed. This will allow QA to have a summary of the changes they are looking at
  • Katrina will review Melissa's cv.ra commits from the previous week, on beta.
  • When Katrina finds issues, Melissa will address the issues both on alpha *and beta*.
  • Once Katrina approves of the changes in beta/cv.ra, she will copy it to public/cv.ra and make public, thereby releasing the changes to the public. The next time trackDb & friends is pushed, the changes to cv.ra will be released. No review of changes to public/cv.ra is necessary since these exact changes were reviewed by Katrina in beta/cv.ra and Melissa in alpha/cv.ra.