Chúng ta nên sử dụng Reject hoặc Wait For Author trong TFS khi có những thứ cần sửa?


12

Trong TFS, khi chúng tôi đưa ý kiến ​​cho những thứ được sửa trong Yêu cầu kéo trước khi chúng tôi chấp nhận, chúng tôi có nên đánh dấu nó là Từ chối hoặc Chờ tác giả không? Cái nào tốt hơn?

nhập mô tả hình ảnh ở đây

Câu trả lời:


14

Theo Microsoft về mã đánh giá với các yêu cầu kéo: Bỏ phiếu thay đổi mục đích đề xuất của từng loại phê duyệt là:

Chấp thuận với các đề xuất : Đồng ý với yêu cầu kéo, nhưng cung cấp các đề xuất tùy chọn để cải thiện mã.

Chờ đợi tác giả : Không chấp thuận các thay đổi và yêu cầu tác giả xem xét nhận xét của bạn. Tác giả nên cho bạn biết khi nào bạn nên xem lại mã sau khi họ đã giải quyết mối quan tâm của bạn.

Bị từ chối : Những thay đổi không được chấp nhận. Nếu bạn đang bỏ phiếu theo cách này, bạn nên để lại nhận xét trong yêu cầu kéo chi tiết tại sao các thay đổi bị từ chối.

Vì vậy, tôi Waiting for Authormuốn nói rằng bạn nghĩ rằng Tác giả đã sai lầm trong cách tiếp cận giải pháp của anh ấy / cô ấy nhưng mã của anh ấy / cô ấy có thể đổi lại được nếu họ lấy ý kiến ​​của bạn.

Rejectedcó nghĩa là không có cách nào trong địa ngục bạn chấp nhận bất kỳ thay đổi như thế này cho dù mã được viết tốt như thế nào.

Câu hỏi bạn phải tự hỏi mình là liệu các nhóm của bạn có ý tưởng về mục đích được đề xuất phù hợp với ý tưởng của Microsoft hay không.


1
Về điều này: "Tác giả nên cho bạn biết khi nào bạn nên xem lại mã sau khi họ đã giải quyết mối quan tâm của bạn." - có một cơ chế để tác giả cảnh báo người đánh giá rằng PR đã sẵn sàng để xem xét lại?
Robert Sim

@RobertSim - chúng tôi sử dụng là tùy chọn 'Ping' bên dưới Người phản biện để gửi tin nhắn yêu cầu xem xét lại.
killercowuk
Khi sử dụng trang web của chúng tôi, bạn xác nhận rằng bạn đã đọc và hiểu Chính sách cookieChính sách bảo mật của chúng tôi.
Licensed under cc by-sa 3.0 with attribution required.