Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

2407 - 3 #287

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

2407 - 3 #287

wants to merge 3 commits into from

Conversation

Pharethi
Copy link
Contributor

@Pharethi Pharethi commented Jul 27, 2018

Номер

2407

Номер задания

3

Ссылка на видео с демо

https://www.youtube.com/watch?v=JwDK22lHDaQ

Комментарии

Выполнил задачу, но отлично понимаю, что просто ужасно.
P.S. Тут выскочила ошибка. которой не было на локалке. Пока что переписываю соответствующий метод.

end

def document
{ 'documents': @texts.map.with_index { |text, index| { 'id' => index.to_s, 'language' => 'ru', 'text' => text } } }

Choose a reason for hiding this comment

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

Lint/Syntax: unexpected token tCOLON
Lint/Syntax: unexpected token tRCURLY

# rubocop:disable Lint/UnneededCopDisableDirective
# rubocop:disable Lint/Syntax
def document
{ 'documents': @texts.map.with_index { |text, index| { 'id' => index.to_s, 'language' => 'ru', 'text' => text } } }

Choose a reason for hiding this comment

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

Lint/Syntax: unexpected token tCOLON
Lint/Syntax: unexpected token tRCURLY

def document
{ 'documents': @texts.map.with_index do |text, index|
{ 'id' => index.to_s, 'language' => 'ru', 'text' => text }
end }

Choose a reason for hiding this comment

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

Lint/Syntax: unexpected token tRCURLY

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants