Skip to content

Conversation

kimjy-par
Copy link
Contributor

Changes

  • Change terminology: '종류' -> '버전'
  • Remove strange characters from string interpolation explanation
  • Fix awkward translations in library documentation

Details

This PR focuses on improving the Korean documentation by correcting terminology, removing unexpected characters, and enhancing overall readability.

@kimjy-par kimjy-par requested a review from a team as a code owner March 15, 2025 07:53
@kimjy-par kimjy-par force-pushed the fix-ko-awkward-translations branch from 4797477 to 7535c15 Compare March 15, 2025 07:58
@kimjy-par kimjy-par force-pushed the fix-ko-awkward-translations branch from 7535c15 to 4a4ef07 Compare March 15, 2025 12:36
@@ -64,7 +64,7 @@ end
매개 변수입니다.

여기서 일어나고 있는 것은 리스트의 각 원소가 `name` 변수에 넘겨져서, 그 `name`을
가지고 `puts “Hello#\{name}!”` 코드가 실행되고 있는 거랍니다.
가지고 `puts “Hello #{name}!”` 코드가 실행되고 있는 거랍니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
가지고 `puts Hello #{name}!` 코드가 실행되고 있는 거랍니다.
가지고 `puts "Hello #{name}!"` 코드가 실행되고 있는 거랍니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the valuable suggestion. I've made the modifications as you recommended.

@@ -65,7 +65,7 @@ Hello Matz!

## 문자열의 중간에 자리를 잡아두기

`#\{name}` 부분은 무엇이냐구요? Ruby에서 문자열 중간에 무엇을 삽입하는 방법입니다.
Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/search?q=repo%3Aruby%2Fwww.ruby-lang.org%20%23%5C%7B&type=code
Code quotes seem to have been added later. There are traces of them in other languages ​​as well.

Copy link
Member

@marocchino marocchino left a comment

Choose a reason for hiding this comment

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

LGTM

@kimjy-par kimjy-par force-pushed the fix-ko-awkward-translations branch from 4a4ef07 to ded4203 Compare March 15, 2025 23:57
@marocchino marocchino merged commit 7cf93ba into ruby:master Mar 16, 2025
1 check passed
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.

3 participants