Skip to content

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

Merged
pepelsbey merged 5 commits intomainfrom
move/array-reduce
Jun 12, 2021
Merged

Переносит статью «Array.reduce»#819
pepelsbey merged 5 commits intomainfrom
move/array-reduce

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

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

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

@nlopin
Copy link
Contributor Author

nlopin commented Jun 6, 2021

Closes #730

@pepelsbey
Copy link
Contributor

pepelsbey commented Jun 6, 2021

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

@pepelsbey
Copy link
Contributor

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

@igsekor
Copy link
Member

igsekor commented Jun 6, 2021

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

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

@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
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.

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

@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
* Переносит статью «Array.reduce»

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

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

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

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

Co-authored-by: Vadim Makeev <pepelsbey@gmail.com>
igsekor pushed a commit that referenced this pull request Jul 22, 2021
* Переносит статью «Array.reduce»

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

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

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

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

Co-authored-by: Vadim Makeev <pepelsbey@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

js Контент по JavaScript статья Расширенный материал

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants