Skip to content

Ensure Migration.new is also included in the transaction #3

@MatheusRich

Description

@MatheusRich

Sort of a weird edge case, but one could theoretically perform db operations inside initializer and that wouldn't be rolled back on failures.

We need to ensure that .new is inside the transaction block. I'd probably move the transaction to self.run.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions