rebuild project structure from scratch / move to poetry

[hnthh]

Задача в бейскемпе: https://3.basecamp.com/5104612/buckets/22530422/todos/6781472420.

[nvo87]

@hnthh привет! Круто, что занялась улучшениями!

Подскажи плииз, а какую проблему решает переезд на струтуру папок с core и apps?

upd: А сорри, если влез на полпути работы в черновик. Мне не срочно, просто интересно. Тегай, когда будет время.

[hnthh]

А покажешь скриншот?

Сейчас уже не так, а до этого было как со Славой сейчас: awaiting requested review from f213. Возможно, такое происходит, когда ревьюер отвечает простыми комментариями в тредах, а не полноценным start review. В гитлабе тоже сплошь и рядом.

[hnthh]

@f213, почти неделя без апдейтов в этом PR. Посмотришь? ;)

[hnthh]

@f213, внесла предварительные правки на основании того, как мне показалось, нужно обработать комментарии.

[nvo87]

я тут напишу, чтобы гитхаб засчитал, что я делал ревью в тот раз. А то так и висю 🤷‍♂️

[nvo87]

🎉🔥🫶 спасибо, Аня и Федя!

[nvo87]

@hnthh привет! Подскажи плииз, а `some_app` так и должна ставиться рядом с src и какое у нее назначение?

[hnthh]

@nvo87, привет!

а some_app так и должна ставиться рядом с src

Если сделать cd src && poetry run python manage.py startapp some_app (т. е. запустить менедж-команду не из корня, а из src), то приложение должно создасться в папке src.

и какое у нее назначение?

Это тестовое приложение, чтобы проверить всё, что мы обычно делаем в бизнесовых проектах, на сепульках (модели, API, тесты, шаблон приложения). После бутстрапа проекта его можно удалить.

Думаю, можно было бы изменить post_gen_project.sh, чтобы запускать стартап-команду сразу из src/ и там же удалять тестовое приложение.

[nvo87]

@hnthh спасибо за комменты!

Если сделать cd src && poetry run python manage.py startapp some_app (т. е. запустить менедж-команду не из корня, а из src), то приложение должно создасться в папке src.

там на самом деле еще больше приколов возникает))

• до этого make lint и make test не проверяли some_app. И даже не знаю, стоит ли пытаться бороться с линтером для свежесозданной аппы.

• some_app создается не по нашему, а по стандартному шаблону. т.е. не подхватывает .django-app-template из кастомного startapp. А это уже критично. Пробовал руками - тоже не работает.

Я смогу глянуть теперь через неделю, что с п.2. Дай знать пожалуйста, если вдруг поймешь в чем дело.

[hnthh]

Тогда надо продолжать запускать команду из корня проекта, а потом либо перемещать созданную папку руками в apps/, либо настроить directory сразу на apps/ в менедж-команде создания приложения.

@nvo87, @f213, могу сделать второе, если нужно.

upd: директорию можно задать руками при создании приложения через directory=, правда, это не так очевидно.

[f213]

могу сделать второе, если нужно.

У нас уже переопределена startapp как раз, чтобы не думать каждый раз об аргументах, так что давай сделаем второе. Заодно, возможно, ты и с неправильным шаблоном разберёшься

[hnthh]

@nvo87, починила ошибки в МР выше.

[nvo87]

@hnthh спасибо большое! Очень понравилось как ты про directory нашла, я и не знал про это 😬