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

Change *kids* to *children* in error message #650

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

blokhin
Copy link

@blokhin blokhin commented Oct 29, 2023

The term kids is very misleading, please, let's use a standard term.

@nin-jin
Copy link
Member

nin-jin commented Nov 3, 2023

А в чём заблуждение? kids существенно более короткий.

@BoolmanO
Copy link

BoolmanO commented Nov 5, 2023

А в чём заблуждение? kids существенно более короткий.

Англоязычному сообществу привычнее термин children, чем kid.
Да и звучит это странно. Ребенок в контексте чей-то (children), и просто ребенок (kid).

@nin-jin
Copy link
Member

nin-jin commented Nov 5, 2023

Есть предложения по более удачному короткому термину?

@blokhin
Copy link
Author

blokhin commented Nov 6, 2023

@nin-jin Дим, правда, не стоит вводить новый термин ради экономии 4 символов, делая сообщение об ошибке для своих коллег при этом совершенно нечитаемым.

@nin-jin
Copy link
Member

nin-jin commented Nov 6, 2023

В сообщении используется тот же термин, что и в коде. А в коде он используется очень часто, что налагает на него требование краткости. kids всяко более понятен, чем используемый ранее sub, и более корректен, чем используемый ещё ранее childs.

@blokhin
Copy link
Author

blokhin commented Nov 7, 2023

@nin-jin вижу, что используется в коде, но здесь речь про информативное сообщение для юзера о том, что что-то не в порядке. У юзера маленький мозг, его нужно пожалеть. Давай компромисс: Children (kids) are not allowed?

@blokhin
Copy link
Author

blokhin commented Nov 13, 2023

вот, пожалуйста, вчера в чате:

1

и такой вопрос наверняка возникает регулярно 🤷‍♂️

@nin-jin
Copy link
Member

nin-jin commented Dec 13, 2023

Боюсь тут проблема не в термине, а в том, что сообщение в принципе не понятно - компилятор ожидает тут стрелочку. @zerkalica сейчас делает рефакторинг компилятора. Будет ли в нём более понятное сообщение?

@blokhin
Copy link
Author

blokhin commented Mar 23, 2024

@zerkalica пожалуйста, напиши свою точку зрения

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