Skip to content

Add note about safety_assured not carrying over during rollback#317

Open
55728 wants to merge 1 commit intoankane:masterfrom
55728:improve-remove-column-rollback-message
Open

Add note about safety_assured not carrying over during rollback#317
55728 wants to merge 1 commit intoankane:masterfrom
55728:improve-remove-column-rollback-message

Conversation

@55728
Copy link
Copy Markdown

@55728 55728 commented Mar 26, 2026

Fixes #284.

When using safety_assured with add_column inside a change method, the safety_assured context does not carry over when Rails reverts the migration. This can be confusing when check_down is enabled, as the rollback hits the remove_column check with no indication of why safety_assured didn't help.

Building on d15e3fa which improved the error output formatting for rollbacks, this adds a brief note to the message content itself suggesting the use of up and down methods instead.

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.

Followed advice; rollback still won't run

1 participant