Skip to content

Update b060027b11b3_eventuser_building.py#130

Merged
petrCher merged 1 commit intomainfrom
petrCher-patch-1
Apr 27, 2026
Merged

Update b060027b11b3_eventuser_building.py#130
petrCher merged 1 commit intomainfrom
petrCher-patch-1

Conversation

@petrCher
Copy link
Copy Markdown
Member

@petrCher petrCher commented Apr 27, 2026

Изменения

Сейчас проблема с деплоем из-за того, что в прошлой миграции уже были удалены поля(те, что я удалил сейчас), почему алембик опять создал в новой миграции удаление уже удаленных полей - непонятно

Детали реализации

В бд проверил, этих полей уже действительно нет, миграция в бд правильная применена

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@github-actions
Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
calendar_backend
   __main__.py660%1–16
   exceptions.py13285%8, 14
calendar_backend/methods
   list_calendar.py704930%28–51, 58–64, 71–76, 85–98, 102–113
   utils.py352626%14–19, 23–28, 34–38, 42–46, 52–56
calendar_backend/models
   base.py64198%87
calendar_backend/routes
   base.py74791%75, 80–81, 86–87, 98, 101
calendar_backend/routes/event
   comment.py47687%30, 41–43, 52, 64
   comment_review.py27485%23–29, 41
   event.py112893%38, 45–48, 53, 113, 117
   user_event.py15753%31–51
calendar_backend/routes/group
   group.py38392%26, 42, 58
calendar_backend/routes/lecturer
   comment.py48883%35, 49, 57–61, 70
   comment_review.py25484%21–29, 41
   lecturer.py60887%27, 41, 44, 54, 78–81
   photo.py43588%43, 48, 65, 76, 79
   photo_review.py512061%39–58, 75–77
calendar_backend/routes/models
   base.py60985%36–40, 43, 56, 67, 80
   event.py62395%15, 42, 60
   group.py391659%13–19, 22, 32–38, 41
   lecturer.py46296%21, 31
   room.py21195%13
calendar_backend/routes/room
   room.py39392%26, 44, 58
TOTAL119819883% 

Summary

Tests Skipped Failures Errors Time
34 0 💤 0 ❌ 0 🔥 4.601s ⏱️

@petrCher petrCher requested a review from Zimovchik April 27, 2026 14:01
@petrCher
Copy link
Copy Markdown
Member Author

@Zimovchik если объяснишь как такое может быть, что я создаю новую миграцию, но в ней применяется удаление которое уже было сделано в прошлой миграции https://github.com/profcomff/timetable-api/blob/main/migrations/versions/55a049fde8f4_many_to_many_events_to_groups.py#L79-L85
p.s. Миграцию сделал Гоша, но я сам локально тоже попытался протестить, та же проблема

@petrCher
Copy link
Copy Markdown
Member Author

По идее после мерджа проблемы в деплое быть не должно

@petrCher petrCher merged commit 1829e2f into main Apr 27, 2026
2 checks passed
@petrCher petrCher deleted the petrCher-patch-1 branch April 27, 2026 14:15
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