Skip to content
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

review chapter 5-8 #414

Merged
merged 15 commits into from
Dec 10, 2019
Merged

review chapter 5-8 #414

merged 15 commits into from
Dec 10, 2019

Conversation

ngdthanhcs
Copy link
Contributor

Nếu bạn tạo Pull Request để dịch, vui lòng hoàn thành các thông tin dưới đây.

  • Đánh dấu [x] (không có dấu cách quanh x) thể hiện dấu tick vào các câu dưới đây cho các câu trả lời 'Có' hoặc 'Đúng'

  • Pull Request này được link tới một issue? Đánh số issue đó ngay sau dấu #: close Revise chương 5-8 #382

  • Bạn có kiến thức về Học Máy không? Chúng tôi ưu tiên những người có trước kiến thức về Học Máy để việc review được thuận tiện hơn.

  • Bạn đã đọc Bảng thuật ngữ chưa? Bạn cần dịch các thuật ngữ theo bảng thuật ngữ chung.

  • Pull request này đã sẵn sàng cho việc review chưa?

Lưu ý

Chỉ bắt đầu review khi mọi mục trên đây đều đã được tick.

ngdthanhcs and others added 3 commits December 4, 2019 15:32
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Copy link
Contributor

@ngcthuong ngcthuong left a 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ài sửa của bạn. Mình có một số comments nhỏ bạn tham khảo thêm.

  • Bạn thêm lại nhiều từ "bạn" và "của bạn" quá. Trong tiếng Việt từ sở hữu cách đôi khi không cần thiết và làm câu nhàm chán và dài. Bạn có thể cân nhắc lại những đoạn sửa trên.

  • Mình nghĩ là không có dấu phẩy trước từ và.

ngdthanhcs and others added 5 commits December 5, 2019 02:19
@tiepvupsu
Copy link
Contributor

tiepvupsu commented Dec 6, 2019

@ngdthanhcs@ngcthuong đẩy nhanh việc merge PR này nhé.

Copy link
Contributor

@ngcthuong ngcthuong left a 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ài dịch của bạn. Còn một số lỗi nhỏ mình sẽ fix và approve mục này.

@ngcthuong
Copy link
Contributor

@tiepvupsu approve mấy suggestion nhỏ hộ mình với.

@tiepvupsu
Copy link
Contributor

@tiepvupsu approve mấy suggestion nhỏ hộ mình với.

Hix, tớ không approve các commit được. Chỉ chủ PR @ngdthanhcs mới accept được.

@ngcthuong
Copy link
Contributor

Không thấy bạn này phản hồi nhỉ. Hay mình approve rồi để sửa sau thêm? @tiepvupsu

ngdthanhcs and others added 2 commits December 10, 2019 10:39
Co-Authored-By: Thuong Nguyen <[email protected]>
Co-Authored-By: Thuong Nguyen <[email protected]>
@ngdthanhcs
Copy link
Contributor Author

ngdthanhcs commented Dec 10, 2019

Không thấy bạn này phản hồi nhỉ. Hay mình approve rồi để sửa sau thêm? @tiepvupsu

mình xin lỗi nhé! Mấy nay quên mất

@ngdthanhcs ngdthanhcs closed this Dec 10, 2019
@ngdthanhcs ngdthanhcs reopened this Dec 10, 2019
@tiepvupsu tiepvupsu merged commit 474fc02 into mlbvn:master Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Revise chương 5-8
4 participants