Skip to content

Latest commit

 

History

History
16 lines (11 loc) · 715 Bytes

INFRA.md

File metadata and controls

16 lines (11 loc) · 715 Bytes
Raw

Infrastructure Reps

  • Team 1: Lin (lhl2617, lhl2617), Yusuf (QFSW, ymi16)
  • Team 2: Eirik (Eirikalb, ea5117)
  • Team 3: Preet (preetl, pl1516), Neelesh (Neelesh99, nr1817)
  • Team 4: James (777jameskim, jhk2217)
  • Team 5: James (JamesTev, jt2220)
  • Team 6: Ning (Diregie-J, nz120)

Infra rules

  1. You do not have access to the main repo's main branch. Make your own branch, and PR into main. You need to have at least one other approval to merge the change into main.

  2. Try to be the one pulling in your team's code, so that you don't review them yourselves. It kind of defeats the code review aspect. Unless circumstances dictate, don't approve your own team's code.

  3. Review PRs in a timely manner.