-
Notifications
You must be signed in to change notification settings - Fork 2
Retrospective notes 2020.04.01
Dominic Oram edited this page Apr 29, 2020
·
2 revisions
This keeps being missed.
- Now added as step to https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Sprint-Planning#preparation
- Instead use shocked face emoji as looking, happy face as approved and sad as rejected.
- Using a spreadsheet is too heavy so remove this
Yes, the demo ticket should include this. We should make sure that more junior members of the team have sufficient input in to doing demos.
- This could be due to people not being confident in how to review - we should make sure that PRs include instructions on how to review
- We could monitor who is owing reviews but this seems a little draconian, John will look into doing it.