rockerest’s avatarrockerest’s Twitter Archive—№ 13,414

  1. …in reply to @slicknet
    slicknet We have a minimum of 2 reviews on code. Commits are irrelevant, a body of work is reviewed via a merge request. The first reviewer cannot merge (by design) and is there to identify obvious problems. The second reviewer (a maintainer) is there to catch nuance and merge.
    1. …in reply to @rockerest
      slicknet There may be may be more reviews: a UX person, a backend person + a frontend person, etc.