Size: a a a

Django [ru] #STAY HOME

2020 July 07

A

Andrey in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
Да, можно
источник

A

Andrey in Django [ru] #STAY HOME
Сейчас сделаю.
источник

MB

Muslim Beibytuly in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
почему сам проект не в контейнере?
источник

A

Andrey in Django [ru] #STAY HOME
Muslim Beibytuly
почему сам проект не в контейнере?
а чо контейнер в каком контейнере?
источник

A

Andrey in Django [ru] #STAY HOME
пхаха
источник

АР

Алексей Рочев... in Django [ru] #STAY HOME
Muslim Beibytuly
почему сам проект не в контейнере?
Не видел в этом особой нужды
источник

A

Andrey in Django [ru] #STAY HOME
так, теперь понял о чем Муслим
я бы в композ такое упаковал
источник

A

Andrey in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
`pycahce`не должно быть в репо
источник

m

moffire in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
Тесты
источник

3_

3ANov _ in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
проект не покрыт тестами - "мы вам перезвоним" )
источник

MB

Muslim Beibytuly in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
1) docker-compose up должно быть достаточно для запуска проекта из коробки
2) Pipfile и тд - лишнее, это иллюзия
3) подключите flake(в идеале почти все из wemake-python-styleguide), bandit+safety+coala+Optional[radon]+mypy
4) pytest подключить и основной флоу покрыть тестами
5) .gitignore из репозитория .gitignore в github
источник

3_

3ANov _ in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
кстати, почему коммитов мало? 🧐
источник

A

Andrey in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
https://github.com/ARoc4ev/django-messanger/blob/master/account/models.py#L55
результат сложения строк и без этого строка
источник

Б

Богдан in Django [ru] #STAY HOME
Алексей Рочев
Добрый вечер! Изучаю django  с надеждой в будущем найти работу. Можете накидать критики ?  https://github.com/ARoc4ev/django-messanger
Привет, про pep8 почитай :)
источник

A

Andrey in Django [ru] #STAY HOME
https://github.com/ARoc4ev/django-messanger/blob/master/account/views.py#L13
https://github.com/ARoc4ev/django-messanger/blob/master/account/views.py#L23
Декоратора нет такого?
если в коробке нет, стоит написать
источник

A

Andrey in Django [ru] #STAY HOME
такая консутрция много где
источник

A

Andrey in Django [ru] #STAY HOME
источник

3_

3ANov _ in Django [ru] #STAY HOME
Богдан
Привет, про pep8 почитай :)
а что там не так по pep8? (вопрос как от новичка)
источник

A

Andrey in Django [ru] #STAY HOME
источник

A

Andrey in Django [ru] #STAY HOME
источник