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

Dịch Intro phần 8 #141

Merged
merged 14 commits into from Dec 23, 2019
Merged

Dịch Intro phần 8 #141

merged 14 commits into from Dec 23, 2019

Conversation

tiepvupsu
Copy link
Collaborator

@tiepvupsu tiepvupsu commented Dec 16, 2019

Pull Request checklist

Đánh dấu tick (x, không có dấu cách xung quanh) 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 introduction/index_vn.md phần 8 #138

  • Bả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?

    • Các phần in nghiêng, in đậm
    • Tiêu đề (số lượng dấu # đầu dòng)
    • Bảng biểu, chú thích cho bảng (dòng phía trên mỗi bảng bắt đầu bằng dấu :)
    • Dịch các từ trong hình vẽ nếu cần. Các dòng có hình có dạng: ![caption](path)
    • Dịch các chú thích hình vẽ (thay các cụm *dịch chú thích ảnh phía trên* bằng bản dịch tiếng Việt)
    • Không dịch các phần code (nằm giữa hai dấu `````)
    • Copy các công thức toán từ bản gốc (các đoạn có $)
    • Giữ các dòng gán nhãn (bắt đầu với :label:)
    • Không tự thêm bớt các dòng trắng
  • 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

  1. Chỉ review nếu Pull Request này đã sẵn sàng cho việc review.

  2. 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:
    img

Copy link
Contributor

@rootonchair rootonchair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phần này em đọc mà thấy gần gũi hẳn :)

chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
tiepvupsu and others added 4 commits December 16, 2019 09:54
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com>
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com>
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com>
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com>
Copy link
Member

@lkhphuc lkhphuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sẽ thảo luận glossary trong #162

chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
Co-Authored-By: Phúc Lê <phuc.lkh@gmail.com>
@thanhcsf thanhcsf added this to Review in progress in v0.14.x Dec 17, 2019
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.

Mình chỉ có một hai comment chính còn các cái khác hơi cá nhân một chút. Cậu xem qua nhé.

Cảm ơn @tiepvupsu vì bài dịch.

chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
chapter_introduction/index_vn.md Outdated Show resolved Hide resolved
@ngcthuong
Copy link
Contributor

Bạn @tiepvupsu vào xem comments nhé

tiepvupsu and others added 7 commits December 19, 2019 20:44
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
Co-Authored-By: Thuong Nguyen <ngcthuong@skku.edu>
v0.14.x automation moved this from Review in progress to Reviewer approved Dec 23, 2019
@aivivn aivivn merged commit b8e3a26 into master Dec 23, 2019
v0.14.x automation moved this from Reviewer approved to Done Dec 23, 2019
@aivivn aivivn deleted the intro_part8 branch December 23, 2019 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v0.14.x
  
Done (Max)
Development

Successfully merging this pull request may close these issues.

Dịch introduction/index_vn.md phần 8
6 participants