New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dịch "chapter_computational-performance/hybridize_vn" - Phần 2 #497
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cảm ơn bạn về bản dịch. Mình có một vài đề xuất, bạn xem qua.
PR này thiếu mất phần đầu của phần 2 rồi nhé |
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
PR này nhờ @rootonchair hoặc @lkhphuc review thêm một approve nữa rồi merge nghen. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mình có mấy comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR sắp approve được rồi 👍
@tamnguyenvan bạn xem qua và resolve PR này nhé. |
Co-Authored-By: Phúc Lê <phuc.lkh@gmail.com> Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Đây có lẽ là đợt review cuối của PR này rồi
@tamnguyenvan commit nốt mấy cái còn lại rồi merge nhé. @duythanhvn tên lặp cũng không sao đâu, thiếu thì mới cần sửa. Sau revise kiểu gì mình cũng phải cleanup lại mục này. |
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com> Co-Authored-By: Thanh Pham Chi <recca2512@gmail.com>
@tiepvupsu em commit rồi ạ. |
Pull Request checklist
Đánh dấu tick (sau khi "Create Pull Request") vào các mục dưới đây
Pull Request này tương ứng với issue nào? Trả lời số issue sau dấu
#
(không có dấu cách): Close Dịch "hybridize_vn" - Phần 2 #482Bản dịch này có bám sát Bảng thuật ngữ không? Nếu một từ/cụm chưa có trong Bảng thuật ngữ, hãy tạo một Pull Request riêng đề xuất cách dịch từ/cụm đó.
Format của bản gốc có được giữ nguyên không?
#
đầu dòng):
)![caption](path)
*dịch chú thích ảnh phía trên*
bằng bản dịch tiếng Việt)$
):label:
)Trong một đoạn văn, mỗi câu văn đã được viết trong một dòng, giữa các dòng không có dòng trắng. Markdown sẽ coi những dòng liền nhau không có dòng trắng là một đoạn văn. Việc này giúp công đoạn review được thuận tiện hơn.
Tất cả chữ trong các hình vẽ đã được dịch. Nếu chưa, vui lòng xem thêm hướng dẫn hoặc yêu cầu dịch hình.
Điền tên của bạn và vào mục "Những người thực hiện" ở cuối file.
Pull Request này đã sẵn sàng cho việc Review chưa?
Sau khi được approve, điền tên những reviewers mà bạn thấy có nhiều đóng góp cho Pull Request này.
Với reviewer
Chỉ review nếu Pull Request này đã sẵn sàng cho việc review.
Bạn có thể đề xuất thay đổi cách dịch mỗi dòng trực tiếp như trong hình dưới đây: