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.

Sign up for GitHub

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

Jump to bottom

Переносит статью «Array.reduce» #819

Merged
merged 5 commits into from
Jun 12, 2021
Merged

Переносит статью «Array.reduce» #819

merged 5 commits into from
Jun 12, 2021

Conversation

nlopin
Copy link
Contributor

@nlopin nlopin commented Jun 6, 2021

No description provided.

@nlopin nlopin requested review from furtivite and pepelsbey June 6, 2021 22:13
@github-actions
Copy link

github-actions bot commented Jun 6, 2021 •
edited
Loading

Превью контента из 798d560 опубликовано.

Время сборки: 6.946 с

@nlopin
Copy link
Contributor Author

nlopin commented Jun 6, 2021

Closes #730

@pepelsbey
Copy link
Contributor

pepelsbey commented Jun 6, 2021 •
edited
Loading

  • Вынес поясняющий текст из примеров кода: до, после, разбил на этапы решения.
  • Перенёс оставшиеся комментарии в коде на свои строки, переформатировал короче
  • Поправил ошибки для спеллера

@pepelsbey
Copy link
Contributor

@igsekor у нас опять поломались тесты, я бы лучше пока откатился и оставил длинные, но надёжные. А потом можно снова вернуть быстрые, как отладим.

@igsekor
Copy link
Member

igsekor commented Jun 6, 2021

@igsekor у нас опять поломались тесты, я бы лучше пока откатился и оставил длинные, но надёжные. А потом можно снова вернуть быстрые, как отладим.

@pepelsbey сижу разбираюсь...

@igsekor igsekor mentioned this pull request Jun 7, 2021
Merged
@igsekor
Copy link
Member

igsekor commented Jun 7, 2021

@pepelsbey сижу разбираюсь...

Посидел, разобрался и сделал PR.

@pepelsbey pepelsbey requested a review from tachisis as a code owner June 8, 2021 16:35
@pepelsbey pepelsbey added js Контент по JavaScript статья Расширенный материал labels Jun 9, 2021
tachisis
tachisis reviewed Jun 9, 2021
View reviewed changes
Copy link
Contributor

@tachisis tachisis left a comment

Choose a reason for hiding this comment

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

Смотрела только изменения в словаре, предлагаю оставить только одну форму по аналогии с «ревьюер»

.yaspeller.json Outdated Show resolved Hide resolved
@nlopin nlopin requested a review from tachisis June 11, 2021 05:34
@pepelsbey pepelsbey merged commit 5ef6b1c into main Jun 12, 2021
@pepelsbey pepelsbey deleted the move/array-reduce branch June 12, 2021 20:28
pepelsbey pushed a commit that referenced this pull request Jun 14, 2021
@nlopin
Переносит статью «Array.reduce» (#819)
3803984 
* Переносит статью «Array.reduce»

* Выносит текст из примеров

* Выносит текст из примеров кода

* Убирает ненужный name

* Удаляет слово редюсер из словаря и статьи

Co-authored-by: Vadim Makeev <[email protected]>
igsekor pushed a commit that referenced this pull request Jul 22, 2021
@nlopin
Переносит статью «Array.reduce» (#819)
718e27e 
* Переносит статью «Array.reduce»

* Выносит текст из примеров

* Выносит текст из примеров кода

* Убирает ненужный name

* Удаляет слово редюсер из словаря и статьи

Co-authored-by: Vadim Makeev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tachisis tachisis tachisis approved these changes

@pepelsbey pepelsbey pepelsbey approved these changes

@furtivite furtivite Awaiting requested review from furtivite

Assignees
No one assigned
Labels
js Контент по JavaScript статья Расширенный материал
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nlopin @pepelsbey @igsekor @tachisis