Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use strong_migrations gem #8730

Merged
merged 3 commits into from Jan 15, 2019
Merged

Use strong_migrations gem #8730

merged 3 commits into from Jan 15, 2019

Conversation

monfresh
Copy link
Contributor

Why: strong_migrations is better maintained than
zero_downtime_migrations and seems to have better advice. The author
of the gem has written many other fantastic gems that are used in
production at Instacart, so they should be trustworthy.

The gem also comes with a handy rake task that will automatically
alphabetize the columns in schema.rb. This will eliminate issues
where after pulling the latest code and running recent migrations
locally, your schema.rb will change.

@ghost ghost assigned monfresh Jan 15, 2019
@ghost ghost added the In-Progress label Jan 15, 2019
@va-bot
Copy link
Collaborator

va-bot commented Jan 15, 2019

1 Warning
⚠️ This is a Big PR. Try to break this down if possible.

Generated by 🚫 Danger

**Why**: `strong_migrations` is better maintained than
`zero_downtime_migrations` and seems to have better advice. The author
of the gem has written many other fantastic gems that are used in
production at Instacart, so they should be trustworthy.

The gem also comes with a handy rake task that will automatically
alphabetize the columns in `schema.rb`. This will eliminate issues
where after pulling the latest code and running recent migrations
locally, your schema.rb will change.
@@ -0,0 +1 @@
StrongMigrations.start_after = 20190111000717
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

@monfresh monfresh added Ready-to-Merge This PR is ready to be merged and will be picked up by va-bot to automatically merge to master and removed In-Progress labels Jan 15, 2019
@ghost ghost added the In-Progress label Jan 15, 2019
@va-bot va-bot merged commit f1b2d7e into master Jan 15, 2019
@va-bot va-bot deleted the mb-strong-migrations branch January 15, 2019 21:20
@ghost ghost removed the In-Progress label Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready-to-Merge This PR is ready to be merged and will be picked up by va-bot to automatically merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants