On Tuesday, March 15, 2016 at 12:12:43 AM UTC-7, Ilja Resch wrote: > > Glad to contribute. But there is still the problem with *text* and > *next. * > ALTER TABLE [test_migration] ALTER COLUMN [text_column] text(2147483647) > NULL > ALTER TABLE [test_migration] ALTER COLUMN [ntext_column] ntext(1073741823) > NULL > > The column size is wrong. > *Text* and *ntext* columns should not have any size behind the column > type. Possible fix would be the addition of: > !["text", "ntext"].include?(type) > in > if [:string, :decimal].include?(sch[:type]) and !["text", "ntext"].include > ?(type) and size = (sch[:max_chars] || sch[:column_size]) > > Thanks >
Sorry, I missed that. That's why it's best if you send a diff instead of just new code. :) I'll commit a fix in a little bit. Thanks for the help! Thanks, Jeremy -- You received this message because you are subscribed to the Google Groups "sequel-talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/sequel-talk. For more options, visit https://groups.google.com/d/optout.
