Review code bằng tiếng Anh cần vừa rõ kỹ thuật, vừa giữ tone cộng tác.
Mục tiêu
- Viết comment không gây công kích
- Phân biệt suggestion và blocker
- Giải thích reason
Soft suggestion
textCould we move this validation into a separate method to make the flow easier to read?
Stronger concern
textI think this may cause a null reference when the customer profile is missing.
Blocker
textThis should be fixed before merging because it changes the existing API contract.
Avoid personal tone
Không viết:
textYou wrote this wrong.
Viết:
textThis logic may not handle the empty response case.
Bài tập nhanh
Viết 3 review comments:
- suggestion
- concern
- blocker