Когда вы должны запускать миграцию по запросу pull?

Question

Извините за такой вопрос noob - новый для git и ruby ​​на рельсах.

Я нахожусь в школьной команде по кодированию, пишу приложение в рельсах. В одном из запросов на загрузку есть новая база данных, но миграции не выполнялись (rake db: migrate).

Должны ли мы сначала выполнить миграцию на ветке, а затем слить, чтобы выполнить master или merge, чтобы выполнить master, а затем выполнить rake db: перенести на главную ветку?

Answer 1

Это лучше практика:

1. запустить rake db:migrate
2. совершить репо

потому db:migrate изменяет свой db/schema.rb файл, который отображает "текущее" состояние базы данных. Очень важно иметь последнюю версию этого файла в вашем репо.

отметить также комментарий в db/schema.rb:

# It's strongly recommended to check this file into your version control system. 

Поэтому рекомендуемый порядок выше.

Answer 2

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

Вот почему я рекомендую проверять ветку, запускать миграции, а затем объединять PR с мастером и нажимать его вверх по течению.

---

Если это ваш запрос тянуть, и вы будете уверены, что все в порядке (или вы работаете в одиночку по проекту), то нет никакого конкретного заказа: просто убедитесь, что вы не забываете все шаги (слияния и мигрировать).

Когда я делаю это сам, у меня обычно уже применяются миграции (потому что я нуждался в них для работы в моей ветке), так что обычно это как раз то, что нужно освоить и объединить ветвь.

Answer 3

Вы должны всегда запускать свои миграции перед слиянием на мастер, потому что вам нужно знать, правильно ли работал этот код, и все в порядке. Помните, что запуск миграции изменит ваш файл db/schema.rb, и если что-то не так, сервер не запустится должным образом или что-то не удастся.

Что касается выполнения миграции до того, как вы передадите ее ветке, это зависит от того, что вы делаете. Если вы - командная кодировка и вам нужно сначала просмотреть работу, продолжайте и совершите, чтобы другие могли проверить филиал и просмотреть.

Answer 4

Почему кто-то открывает запрос на перенос на ветке с новыми миграциями, которые еще не были запущены? Как автор запроса на растяжение знает, что миграция была написана правильно?

Если бы я просматривал запрос на перенос, я ожидал, что миграции будут выполняться до слияния ветви с хозяином. Кроме того, убедитесь, что ваш schema.rb получает проверку с миграциями.