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

vi: Control flow basics summary #2074

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

HuyNVuong
Copy link
Contributor

I prefer using "Lệnh" over "Biểu thức" because AFAIK Biểu thức refer to RHS expression things such as a + b or 100 if a == x else 200.... So Lệnh seems like a better term in the header
Correct me if I'm wrong

Copy link
Contributor

@vohoanglong0107 vohoanglong0107 left a comment

Choose a reason for hiding this comment

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

I agree that Lệnh works better in this context.

po/vi.po Outdated Show resolved Hide resolved
@@ -111,11 +111,11 @@ msgstr ""

#: src/SUMMARY.md src/control-flow-basics.md
msgid "Control Flow Basics"
msgstr ""
msgstr "Điểu khiển luồng căn bản"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think Control flow can be used as is here. Reader can get a rough understanding in the Glossary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think Control flow seems like a hard word to spell for non-English speakers right? So I think keeping the translations here for instructor / students who have a hard time of spelling it refer to without the needs of referrring to the glossary is good. Maybe Control flow (Điểu khiển luồng) căn bản?

po/vi.po Outdated Show resolved Hide resolved
@mgeisler mgeisler changed the title Control flow basics summary vi: Control flow basics summary May 21, 2024
@HuyNVuong HuyNVuong requested a review from qu-ngx as a code owner May 31, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants