Skip to content

Conversation

@jiyongp
Copy link
Collaborator

@jiyongp jiyongp commented Jan 30, 2023

  • add ko.po
  • translate ko(~23.01.19)
  • change speaker-note ko
  • change id
  • translate ko(~23.01.20)
  • ~day3 keynote
  • draft done to f3446a9
  • add @jiyongp comments of upstream PR Initial Korean commit #276.
  • sync & apply review comments(upstream)
  • After 10000 line apply review comments(upstream)
  • chgange build.yml
  • Fix the inconsistent newline character problem for the KO translation

@jiyongp jiyongp changed the title Fix incorrect newline problem for the KO translation Add Korean translation Jan 30, 2023
@jiyongp
Copy link
Collaborator Author

jiyongp commented Jan 30, 2023

This is basically #290 from @keispace. The only change from me is the fix of the inconsistent newline character problem. I've created a new PR because I couldn't amened the original PR.

@jiyongp jiyongp marked this pull request as ready for review January 30, 2023 05:11
@jiyongp jiyongp merged commit 1040115 into google:main Jan 30, 2023
This was referenced Jan 30, 2023
@mgeisler mgeisler mentioned this pull request Jan 30, 2023
NoahDragon pushed a commit to wnghl/comprehensive-rust that referenced this pull request Jul 19, 2023
* add ko.po

* translate ko(~23.01.19)

* change speaker-note ko

* change id

* translate ko(~23.01.20)

* ~day3 keynote

* draft done to f3446a9

* add @jiyongp comments of upstream PR google#276.

* sync & apply review comments(upstream)

sync 585509b

* After 10000 line apply review comments(upstream)

* chgange build.yml

* Fix the inconsistent newline character problem for the KO translation

If a msgid does not end with the newline character, so should the
msgstr.

Test: msgfmt --statistics -o /dev/null po/ko.po
No error, but shows `1085 translated messages, 675 untranslated
messages.`

---------

Co-authored-by: Evan kim(cli) <[email protected]>
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.

3 participants