Skip to content

allow ignoring comments in migration checksum #3909

@siosw

Description

@siosw

I have found these related issues/pull requests

/

Description

currently migration checksums include comments, this makes it hard to change them once a migration has been applied. sometimes this is nice to add more context later etc.

Prefered solution

add some logic filtering out comments here:

let sql = fs::read_to_string(&entry_path).map_err(|e| ResolveError {

Is this a breaking change? Why or why not?

this would break existing migration checksums so it should be behind a feature flag

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions