VA
Size: a a a
2020 February 14
VA
Нашел
AV
Непонятно как твой код запустить. Нужно добавить в ридми и сделать какой-то dockerfile или makefile
M
Всем привет. Недавно делал тестовое на должность python разработчика.
https://github.com/VetrovCNC/suggests-scraper-alloua
У кого есть сильное желание покритиковать код буду весьма признателен.
https://github.com/VetrovCNC/suggests-scraper-alloua
У кого есть сильное желание покритиковать код буду весьма признателен.
https://github.com/VetrovCNC/suggests-scraper-alloua/blob/master/config.py
Я бы не использовал env vars для этого, либо аргументы командной строки либо текстовый файл
Я бы не использовал env vars для этого, либо аргументы командной строки либо текстовый файл
M
если это не делается одной командой это вызывает вопросы
AV
если это не делается одной командой это вызывает вопросы
если в вашем окружении доступны эти зависимости (requests, SQLAlchemy) то просто запустить scraper.py.
IS
Vitality Androsenko
Спасибо. Гугл мне выдал статью на хабре)
M
если в вашем окружении доступны эти зависимости (requests, SQLAlchemy) то просто запустить scraper.py.
А почему просто не сделать Dokerfile?
VA
Ага. Т.е. про setup.py ты пока не в курсе. Есть шанс сегодня узнать много нового :)
VA
Спасибо. Гугл мне выдал статью на хабре)
Ох и не люблю же я хабр. Видос получше будет.
AV
А почему просто не сделать Dokerfile?
Можно сделать. Но это же было тестовое и в вакансии докер не упоминался, поэтому и опустил. А с виртуальным окружением тоже норм. запускается.
YM
Можно сделать. Но это же было тестовое и в вакансии докер не упоминался, поэтому и опустил. А с виртуальным окружением тоже норм. запускается.
не совсем очевидно, как это запускать
переименовать scraper.py на main.py хотя бы
или добавить в README как запускать
переименовать scraper.py на main.py хотя бы
или добавить в README как запускать
AV
не совсем очевидно, как это запускать
переименовать scraper.py на main.py хотя бы
или добавить в README как запускать
переименовать scraper.py на main.py хотя бы
или добавить в README как запускать
Да. нужно будет инструкцию по запуску написать.
MO
Главное чтобы рекрутеры в этом чате не сидели, вряд ли они оценят коллективный разум ))
AV
Главное чтобы рекрутеры в этом чате не сидели, вряд ли они оценят коллективный разум ))
компания куда я делал это задание уже его проверила и было собеседование. Я просто хотел получить мнение других разработчиков. И весьма за это благодарен.
DL
компания куда я делал это задание уже его проверила и было собеседование. Я просто хотел получить мнение других разработчиков. И весьма за это благодарен.
а почему ты треды юзал а не asyncio?
DF
А в сколько потоков он скрапит там?
YM
а почему ты треды юзал а не asyncio?
но это почему бы и нет
AV
Dollar Føølish
А в сколько потоков он скрапит там?
по умолчанию в 10, но можно переопределить.
AV
а почему ты треды юзал а не asyncio?
так мне показалось проще.