General | Org repo index that helps people find solutions, says which repos are maintained, which repos aren't recommended any more Interchange examples update -- Make sure this is on to do, update Thompson on status Decision: We’ll keep Interchange user guide in Interchange, give JM direct access for small changes, clear guidelines for medium/large changes Minimum requirements (can always request review and owner will provide it): Small changes (fix typo, wording change of 10 lines or less) → Feel free to merge directly once status is green Medium changes (meanings change but unambiguous improvement) → Give 3 working days after owner is tagged for review, if time window passes with no review or no “I need more time” message, then merge. Large changes (functionality or significant code change) → Requires review
JW – We’re also going to do two things: When we review a PR from someone who has direct merge access, we’ll say “here’s whether I think this really required review, and here’s how I could tell” We’re aiming to do a NON ZERO number of PR reverts this year for each other’s work. If we get to the end of 2022 and haven’t reverted any PRs, then we know we’re being too cautious.
|