Improve CLI for launching and managing the database #154

Merged
kudyakovaa merged 7 commits from t3m8ch/improving-cli into main 2025-04-15 11:51:08 +00:00
kudyakovaa commented 2025-04-08 09:54:04 +00:00 (Migrated from git.csit.sgu.ru)

Closes #17

Closes #17
kudyakovaa commented 2025-04-08 09:54:04 +00:00 (Migrated from git.csit.sgu.ru)

requested review from @ProstoLive

requested review from `@ProstoLive`
kudyakovaa commented 2025-04-08 09:54:04 +00:00 (Migrated from git.csit.sgu.ru)

requested review from @nrydanov

requested review from `@nrydanov`
kudyakovaa commented 2025-04-08 12:51:51 +00:00 (Migrated from git.csit.sgu.ru)

changed title from {-[WIP] -}Improve CLI for launching and managing the database to Improve CLI for launching and managing the database

changed title from **{-[WIP] -}Improve CLI for launching and managing the database** to **Improve CLI for launching and managing the database**
rudyakas commented 2025-04-08 13:08:28 +00:00 (Migrated from git.csit.sgu.ru)

Review: Approved

Good

**Review:** Approved Good
rudyakas commented 2025-04-08 13:08:28 +00:00 (Migrated from git.csit.sgu.ru)

approved this merge request

approved this merge request
rydanovns commented 2025-04-08 17:48:29 +00:00 (Migrated from git.csit.sgu.ru)

На самом деле нет большого смысла игнорить одиночные файлы, они ничего не весят

На самом деле нет большого смысла игнорить одиночные файлы, они ничего не весят
rydanovns commented 2025-04-08 17:49:36 +00:00 (Migrated from git.csit.sgu.ru)

Можно 1.24

Можно 1.24
kudyakovaa commented 2025-04-09 04:38:51 +00:00 (Migrated from git.csit.sgu.ru)

Ну db я заигнорил ещё потому, что он монтируется в контейнер с хостовой машины. .env для безопасности надо игнорить. На счёт остальное, в целом согласен, но ничего плохого не будет, если там эти файлы останутся. Надо ещё сюда tmp запихать, который air генерирует, вот его точно в образ не надо пихать, тем более он тоже монтируется с хостовой машины

Ну db я заигнорил ещё потому, что он монтируется в контейнер с хостовой машины. .env для безопасности надо игнорить. На счёт остальное, в целом согласен, но ничего плохого не будет, если там эти файлы останутся. Надо ещё сюда tmp запихать, который air генерирует, вот его точно в образ не надо пихать, тем более он тоже монтируется с хостовой машины
kudyakovaa commented 2025-04-09 04:39:09 +00:00 (Migrated from git.csit.sgu.ru)

Хорошо, обновлю!

Хорошо, обновлю!
kudyakovaa commented 2025-04-09 04:40:38 +00:00 (Migrated from git.csit.sgu.ru)

Можно в целом и Postgresql обновить с 17.1 до 17.4. Просто я не уверен, что это нужно в этом PR делать. Возможно, стоит это как-то отдельно закоммитить. Там, кстати, гх ещё орёт, что уязвимости есть. Их тоже бы фиксануть.

Можно в целом и Postgresql обновить с 17.1 до 17.4. Просто я не уверен, что это нужно в этом PR делать. Возможно, стоит это как-то отдельно закоммитить. Там, кстати, гх ещё орёт, что уязвимости есть. Их тоже бы фиксануть.
kudyakovaa (Migrated from git.csit.sgu.ru) closed this pull request 2025-04-15 11:51:08 +00:00
kudyakovaa (Migrated from git.csit.sgu.ru) merged commit into main 2025-04-15 11:51:08 +00:00
rudyakas (Migrated from git.csit.sgu.ru) approved these changes 2026-06-15 07:10:16 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
dsc/mm-backend!154
No description provided.