Skip to content

Conversation

@mecmep
Copy link
Contributor

@mecmep mecmep commented Aug 25, 2023

Добавлена типичная ошибка отсутствия после значения в JSON запятой. Также, возможно, стоит добавить ошибку с избыточной запятой после значения.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Aug 25, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team August 25, 2023 19:05
@CLAassistant
Copy link

CLAassistant commented Aug 25, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Alexandre887 Alexandre887 left a comment

Choose a reason for hiding this comment

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

Неплохие идеи, особенно касательно запятой после последнего значения. Это, наверно, самая распространённая ошибка

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Aug 26, 2023
@javascript-translate-bot
Copy link
Contributor

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@mecmep mecmep changed the title Добавлена типичная ошибка отсутствия запятой Добавлены типичные ошибки отсутствующей и лишней запятой в JSON Aug 28, 2023
@mecmep
Copy link
Contributor Author

mecmep commented Aug 28, 2023

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Aug 28, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team August 28, 2023 10:55
@javascript-translate-bot javascript-translate-bot added needs +1 One more review needed and removed review needed Review needed, please approve or request changes labels Aug 28, 2023
@Alexandre887 Alexandre887 merged commit 178d7c9 into javascript-tutorial:master Aug 28, 2023
@mecmep mecmep deleted the mecmep-patch-1 branch August 30, 2023 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs +1 One more review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants