Sean Chambers

Results 18 comments of Sean Chambers

@jzabroski, let me know if you'd like for me to open a PR: https://github.com/fluentmigrator/fluentmigrator/commit/7da19ee30e04ff49dd2414a14993fed77d5fc4e2

For your review @jzabroski https://github.com/fluentmigrator/fluentmigrator/pull/1744 If you approve feel free to merge or I can merge later today. I've applied the 5.2.0 milestone to this fix as well

Issue resolved and merged to main. fyi @volkanceylan

I think if anything, certain databases may throw an error if attempted to combine unique constraint and primary key on same column but perhaps we determine if thats the case...

I can take this. It's easy and I've set this up at the last couple of workplaces

Hmm. there is a method of grouping updates so that there's one PR for multiple updates. Are you referring to the other issue that had to do with dependency updates...

I think I'll ignore the "groups" feature and wait for the new feature like you mentioned @jzabroski. I think after the new feature is released we'll determine if the "groups"...

Thanks for the issue @rhegner. This seems to be a duplicate of #1737 and is awaiting merge as part of PR #1744. If you'd like to verify, feel free to...