Skip to content

Conversation

@Snezhnyy
Copy link

@Snezhnyy Snezhnyy commented Apr 16, 2018

Фамилия Имя

Добшиков Анатолий

Email

anatoly.dobshikov@mail.ru

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

2

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

https://youtu.be/Wpzg-VE5PXE

Комментарии

Бот работает с postgresql через active record; с api телеграма через telegram-bot-ruby; с api github через octokit и через curl. Что-то непонятное и страшное было с методом api git'a search/commits - он не хотел видеть paging от меня, а так мне все понравилось. Видео долгое получилось, но музыка скрасит время, проведенное за его просмотром.
Ссылка на бота: t.me/Cindy_GitHelper_Bot. Вдруг кто захочет поиграться, а у меня будет запущен клиент)
Валидация репозиториев присутствует, но по базе данных из видео можно подумать, что это не так. Это связано с тем, что я не дропнул таблицу после нескольких тестов, к которым я подключал всех желающих (т.е. заставлял силой всех, до кого дотянулся).

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

.rubocop.yml: Lint/BlockAlignment has the wrong namespace - should be Layout
.rubocop.yml: Lint/BlockAlignment has the wrong namespace - should be Layout
.rubocop.yml: Lint/DefEndAlignment has the wrong namespace - should be Layout
.rubocop.yml: Lint/EndAlignment has the wrong namespace - should be Layout
Error: The `Style/TrailingCommaInLiteral` cop no longer exists. Please use `Style/TrailingCommaInArrayLiteral` and/or `Style/TrailingCommaInHashLiteral` instead.
(obsolete configuration found in .rubocop.yml, please update it)
obsolete parameter MaxLineLength (for Style/IfUnlessModifier) found in .rubocop.yml
`Style/IfUnlessModifier: MaxLineLength` has been removed. Use `Metrics/LineLength: Max` instead
obsolete parameter MaxLineLength (for Style/WhileUntilModifier) found in .rubocop.yml
`Style/WhileUntilModifier: MaxLineLength` has been removed. Use `Metrics/LineLength: Max` instead

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.

2 participants